Skip to content
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 JVM options in Dockerimage configurable #344

Closed
ceari opened this issue Aug 27, 2024 · 0 comments · Fixed by #346
Closed

Make JVM options in Dockerimage configurable #344

ceari opened this issue Aug 27, 2024 · 0 comments · Fixed by #346
Assignees
Labels
enhancement New feature or request

Comments

@ceari
Copy link

ceari commented Aug 27, 2024

We are currently deploying the feasibility-backend in Kubernetes and would like to configure some JVM options to tune the maximum heap size via -XX:MaxRAMPercentage , for example.

However, the Dockerimage does not allow configuring JVM options currently.

It would be useful to be able to pass JVM options via JAVA_OPTS environment variable (or similar) to the java command in the entrypoint script.

See https://github.com/medizininformatik-initiative/feasibility-backend/blob/v5.0.1/docker-entrypoint.sh#L31 (and line 34)

michael-82 added a commit that referenced this issue Aug 29, 2024
- add $JAVA_OPTS env variable to java call in docker-entrypoint.sh
@michael-82 michael-82 self-assigned this Aug 29, 2024
@michael-82 michael-82 added the enhancement New feature or request label Aug 29, 2024
@michael-82 michael-82 linked a pull request Aug 29, 2024 that will close this issue
michael-82 added a commit that referenced this issue Sep 2, 2024
- add $JAVA_OPTS env variable to java call in docker-entrypoint.sh
juliangruendner added a commit that referenced this issue Sep 3, 2024
…-make-jvm-options-in-dockerimage-configurable

#344 - Make JVM options in Dockerimage configurable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants