Skip to content

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.

3 participants