Skip to content

Comments

Add reflection parameters to remote server startups#989

Merged
labkey-jeckels merged 2 commits intodevelopfrom
fb_jvmArgsForPipelineServer
Feb 15, 2025
Merged

Add reflection parameters to remote server startups#989
labkey-jeckels merged 2 commits intodevelopfrom
fb_jvmArgsForPipelineServer

Conversation

@labkey-jeckels
Copy link
Contributor

Rationale

Remote pipeline servers need these JVM arguments too

Changes

  • Add the required --add-opens arguments

@labkey-jeckels labkey-jeckels requested a review from a team February 13, 2025 17:45
@labkey-jeckels
Copy link
Contributor Author

@labkey-adam you're on Windows. Can you try running installServiceWin.bat? I think I've got the syntax correct but don't have an easy way to test locally.

@labkey-adam
Copy link
Contributor

@labkey-adam you're on Windows. Can you try running installServiceWin.bat? I think I've got the syntax correct but don't have an easy way to test locally.

I ran it and it seemed to execute successfully, installing "LabKey Pipeline Remove Service". Starting the service failed; the log just says, "The system cannot open the file." I'm not sure what state my LabKey deployment needs to be in for this to work.

@labkey-jeckels
Copy link
Contributor Author

@labkey-adam you're on Windows. Can you try running installServiceWin.bat? I think I've got the syntax correct but don't have an easy way to test locally.

I ran it and it seemed to execute successfully, installing "LabKey Pipeline Remove Service". Starting the service failed; the log just says, "The system cannot open the file." I'm not sure what state my LabKey deployment needs to be in for this to work.

OK, thanks. That's probably good enough for quick validation purposes.

@labkey-jeckels labkey-jeckels merged commit 686918f into develop Feb 15, 2025
3 checks passed
@labkey-jeckels labkey-jeckels deleted the fb_jvmArgsForPipelineServer branch February 15, 2025 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants