Skip to content

GSH_JVMARGS not taken from environment #38

Closed
ghost opened this issue Apr 26, 2018 · 5 comments
Closed

GSH_JVMARGS not taken from environment #38

ghost opened this issue Apr 26, 2018 · 5 comments
Labels

Comments

@ghost
Copy link

ghost commented Apr 26, 2018

On line 7 in the gsh wrapper, the export command for GSH_JVMARGS doesn't include any existing value from that environment variable. Even if we set GSH_JVMARGS in the Dockerfile, the value is overwritten when this script executes. Perhaps adding ${GSH_JVMARGS} to the export command would help. This was also noticed in line 7 of the daemon wrapper.

@ghost
Copy link
Author

ghost commented Apr 27, 2018

Hi @kwessel,

Are you talking about the gsh file in /usr/local/bin or the gsh file in /opt/grouper/grouper.api/bin?

@ghost
Copy link
Author

ghost commented Apr 27, 2018

I guess I should add. I think this is a @chyzer issue as the files in /usr/grouper/grouper.api/bin are "his" files.

@ghost ghost self-assigned this Apr 27, 2018
@ghost
Copy link
Author

ghost commented May 3, 2018

No, this isn't the Grouper native scripts, this is the TIER Grouper wrappers in /usr/local/bin. We tried to associate this issue specifically with /usr/local/bin/daemon and /usr/local/bin/gsh, but I guess that didn't work. It should be a trivial change to those two wrapper scripts, though.

@ghost ghost added the bug label May 8, 2018
@ghost
Copy link
Author

ghost commented May 8, 2018

Oh, gotcha. Sorry I completely forgot that I has set that variable to enable logging. I've included a self-reference to the variable when setting it in the gsh and daemon scripts. It's being built into master now. It'll get pulled into the next patch-base tag when the Grouper releases the next set of patches. (Unless you'd like us to pull the change into an older tag... let me know which if so.)

@ghost
Copy link
Author

ghost commented May 8, 2018

The master builds is complete. Please give it a test and confirm that it meets your needs. Or if you need me to apply the change to a specific grouper patch set.

@ghost ghost closed this as completed May 8, 2018
This issue was closed.
Sign in to join this conversation on GitHub.
Labels
Projects
None yet
Development

No branches or pull requests

0 participants