-
Notifications
You must be signed in to change notification settings - Fork 80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make startup command configurable #505
Conversation
e0f8f4f
to
9f684c1
Compare
If we go with this approach where the whole command is configurable.. which is fine by me .. we should update commit message, PR name, release notes suggestion, and also add relevant info to the docs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See earlier comment of mine...
9f684c1
to
9dc4bc1
Compare
f5f0ecc
to
4b0d3ca
Compare
8624f4b
to
bbe2c15
Compare
Co-authored-by: Manfred Moser <[email protected]>
bbe2c15
to
ad9f223
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now. Thank you @willmostly
Description
Make the startup command configurable
Additional context and related issues
This allows users to set java options such as
-Djavax.net.ssl.truststore=/path/to/truststore
, run short scripts, and otherwise customize startup.Release notes
required.
Helm Chart
Make the startup command configurable