-
Notifications
You must be signed in to change notification settings - Fork 12
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
gateway - run the georchestra gateway with a Java runtime 21 (fixes geor/geor#4285) #139
base: master
Are you sure you want to change the base?
Conversation
Hopefully a JRE 21 will be available in the next debian release, but waiting for it, we can set up a tmurin-21-jre from adoptium, see georchestra/georchestra#4285 Tests: * rspec ok after having installed exim4-base and relaunch the DF * runtime tests ok: weird redirection to CAS, but able to log in by hitting /login directly (through the georchestra gateway).
there's another reference to the default java version in the df: installing temurin 21 will change the default java version .. no ? does the df backend work with java 11 ? 17 ? 21 ? any of them ? |
Hmm, right :-/
I think we should avoid the update-alternatives step here, and I'd like to only affect the gateway, not the other apps. |
it works with both java 17 & 21 in my vagrant. |
I checked if there was a way to avoid temurin-21 to become the default jre, but the postinst script does not allow it, so maybe playing with the alternatives ansible module after having setting it up ? e.g.: For the record, here is the postinst provided by the temurin package:
|
i'm totally fine with using the alternatives module, i have that somewhere:
the only question left is 'should this be done for all binaries provided by the jre/jdk'... |
I guess so, my fear was to have more tools from one side or another (between java21 vs java17). |
there doesnt seem to be a dpkg/apt option to avoid running postinst scripts... |
I have seen a hackish solution on stackoverflow, but I did not want to go in that direction (exit 0 in a /etc/apt/something.d directory where you can put hooks) |
Hopefully a JRE 21 will be available in the next debian release, but waiting for it, we can set up a temurin-21-jre from adoptium, see georchestra/georchestra#4285
Tests: