-
Notifications
You must be signed in to change notification settings - Fork 166
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
Jenkins will require Java 11 from September LTS release #2984
Comments
Looking at the release schedules (nodejs/Release#737, nodejs/Release#658, nodejs/Release#567) I'm going to plan to do the updates on 25 Aug 2022, with the proviso that if a security release is needed/happens during August that requires a CI lockdown I would shift doing the updates to between after the security release and before lifting the CI lockdown. |
Maintenance has started. I've shutdown the test Jenkins CI, powered off the server and am in the process of taking a snapshot in DigitalOcean ( |
As a (self-)reminder: just before shutting down the server I unselected all "build" (can start builds) permissions to prevent builds from being started during the updates and will need to put them back when the maintenance is over. |
I've managed to upgrade the server from Ubuntu 16.04->18.04->20.04->22.04.1. Had a nameserver issue, which we've run into before with DO droplets and required the same fix as #2745 (comment). Working on switching to Java 17 now. |
Also the initial update to Ubuntu 18.04 still had the server on a very old kernel:
There's a "Kernel" page for the droplet in DigitalOcean which I switched to "Grubloader" and after a restart the server then used the expected kernel for the Ubuntu release. We're now on:
|
Have had to apply afonsof/jenkins-material-theme#193. It looks like the theme were using has some issues with the latest LTS Jenkins release 😞 . |
Run into an issue with Java 17 -- when I start Jenkins with it, attempting to start a build that uses our VersionSelectorScript.groovy is now erroring:
49 and 57 are ASCII codes for "1" and "9", e.g. "19". Problem goes away (i.e. job successfully starts) if I switch back to Java 8 and restart Jenkins 😞 . |
Going to try Java 11 for comparison. |
Nope, same error with Java 11 😞. |
Looks like we're running into jenkinsci/matrix-groovy-execution-strategy-plugin#20 😞 . |
Looks like the current LTS version of Jenkins and the custom theme we have don't play well together -- black text on black background for console output in jobs and several missing icons. I've removed our custom theme for now. |
The Matrix Groovy Execution Strategy plugin appears to have an issue on Java 11+ where string parameters are appearing as byte arrays instead of strings. Use the new String() constructor instead of toString() to ensure we get string repesentation of the parameter. Refs: jenkinsci/matrix-groovy-execution-strategy-plugin#20 Refs: #2984 (comment)
Implemented a workaround in #3019. |
Summary:
|
Great work @richardlau, thank you! |
We probably need to start planning for this:
https://www.jenkins.io/blog/2022/06/28/require-java-11/
Both CI servers are currently running Java 8.
cc @nodejs/build-infra
The text was updated successfully, but these errors were encountered: