-
Notifications
You must be signed in to change notification settings - Fork 167
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
Upgrade java used for the jenkins CI server from 8u202 #2492
Comments
Yeah, probably a good idea, but we've learned to be very hesitant about any changes to Java in our Jenkins stack because we've had such a terrible history with Java-related problems being thrown up at random - when you have something stable, don't touch it! is the mentality. It really needs someone dedicated to dealing with potential fall-out from such an upgrade. Otherwise we're just consuming what the OS is feeding us. |
Given the Jenkins servers are running on Ubuntu 16.04 and the end of LTS for that is the end of this month we should start planning on updates, including this. |
This issue is stale because it has been open many days with no activity. It will be closed soon unless the stale label is removed or a comment is made. |
We're now running on Java 17, see #2984. |
I was just looking at the java that hosts ci.nodejs.org and it seems to be Oracle's 8u202 (the last version before Oracle started restricting the licensing of java 8)
Since this version is now almost two years old I would recommend upgrading it to keep secure, and in the spirit of open source I would reocmmend using a version sourced from AdoptOpenJDK instead of Oracle (Disclaimer - I work at AdoptOpenJDK).
We have discussed this previously in issue #1222 that went stale, but I think we should seriously consider revisiting this and upgrading the CI hosts as a minimum to a more recent 8 (or later version if desired). Using the OpenJ9 variant instead of HotSpot would potentially have memory benefits too. Happy to assist in any way I can of course :-)
The text was updated successfully, but these errors were encountered: