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

Upgrade java used for the jenkins CI server from 8u202 #2492

Closed
sxa opened this issue Dec 4, 2020 · 4 comments
Closed

Upgrade java used for the jenkins CI server from 8u202 #2492

sxa opened this issue Dec 4, 2020 · 4 comments

Comments

@sxa
Copy link
Member

sxa commented Dec 4, 2020

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 :-)

@rvagg
Copy link
Member

rvagg commented Dec 7, 2020

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.

@richardlau
Copy link
Member

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.

@github-actions
Copy link

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.

@github-actions github-actions bot added the stale label Feb 13, 2022
@mhdawson mhdawson removed the stale label Feb 21, 2022
@richardlau
Copy link
Member

We're now running on Java 17, see #2984.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants