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

JVM args moved to JAVA_OPTS envvar (instead of JAVA_OPTS_APPEND) #186

Merged

Conversation

guidograzioli
Copy link
Member

@guidograzioli guidograzioli commented Apr 8, 2024

The role parameter keycloak_quarkus_java_opts now contains all arguments passed concatenated to the JVM, and it is used to value the JAVA_OPTS environment variable read be kc.sh.

By dafault, it takes the contatenation of two new arguments:

Variable Description Default
keycloak_quarkus_java_opts Heap memory JVM setting -Xms1024m -Xmx2048m
keycloak_quarkus_java_jvm_opts Other JVM settings same as keycloak
keycloak_quarkus_java_opts JVM arguments; if overriden, it takes precedence over keycloak_quarkus_java_* {{ keycloak_quarkus_java_heap_opts + ' ' + keycloak_quarkus_java_jvm_opts }}

Fix #185

@guidograzioli guidograzioli added the bugfixes Fixes that resolve issues. SHOULD not be used for minor enhancements label Apr 8, 2024
@guidograzioli guidograzioli merged commit 10d4cb8 into ansible-middleware:main Apr 9, 2024
18 checks passed
@guidograzioli guidograzioli changed the title JVM arguments go in JAVA_OPTS JVM arguments moved to JAVA_OPTS envvar (instead of JAVA_OPTS_APPEND) Apr 17, 2024
@guidograzioli guidograzioli changed the title JVM arguments moved to JAVA_OPTS envvar (instead of JAVA_OPTS_APPEND) JVM args moved to JAVA_OPTS envvar (instead of JAVA_OPTS_APPEND) Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfixes Fixes that resolve issues. SHOULD not be used for minor enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

duplicate keycloak_quarkus_java_opts (Xms Xmx) used
1 participant