-
-
Notifications
You must be signed in to change notification settings - Fork 101
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
Ansible request for ensuring jenkins agents are run with JDK11+ #2763
Comments
So far none of the above options are proving to be feasible for Solaris 10 due to the dependency on the Solaris 11 |
We should aim for JDK17 on all systems (For AIX we can use a nightly build or GA candidate) |
Fixed. a JDK11 with a 'fake'
A couple of extra adjustments were needed to make
And set the jenkins configuration for the agent to point to |
This looks like a good solution to me, the only real impact is on the jenkins agent, its probably a cleaner (and less prone to errors) than running via ssh forwarding or similar. |
Unfortunately I hadn't noticed that the linked job ran on the old Solaris machine (I thought it had been disabled) so this isn't fully working yet as the
|
Considering not persuing this and going down another route - build JDK11 on a Solaris 11 system but adjust the code so it doesn't require |
@ptribble's 11.0.2 works on Solaris 10/x64: https://pkgs.tribblix.org/openjdk/openjdk11.0.2-s10-x86_64.tar.bz2 |
Using LD_PRELOAD_64 rather than a bare LD_PRELOAD ought to restrict it to 64-bit processes, if you want to pursue that route. |
Thank you! I don't think I've used that before but it's EXACTLY what I needed and https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-solaris-x64-temurin/208/ has successfully built with the job running via a JDK11u jenkins agent on the machine. So, for reference:
There were a few messages showing in the startup log on SPARC, but that appears to be building ok too. FYI @speakjava - we might have a solution :-) |
We still have over 50 systems running JDK8 as the agent. Thanks to @steelhead31 for collating this list. We can check these off as they are fixed. Also need to ensure that the playbooks deploy new ones with JDK11/17 available (and ideally as the default) Windows (14)
Linux build + test (22)
Others (16)
Note that from the comments in #2847 the Solaris 10 machines will require Bellsoft Liberica 11 to be used (Azul's seems to result inhigh CPU load after a while on Solaris 10) and requires the |
@steelhead31 @Haroon-Khel I've just adjusted the last comment to categorise the machines (Windows/Linux build+test/Other). I reckon at this point we should set an hour aside and do this. Would you be interested in doing it with an open call to discuss any issues? FYI @karianna in case you have someone who has access and would like to take on the windows subset :-) @steelhead31 Can you easily tell how many of the ones already migrated are using a jenkins config that points to a specific java as opposed to the default version? (I'm not sure if the command lines you were getting would say just |
@sxa / @Haroon-Khel sounds like a good idea to me, I'll produce an up to date ( and complete list ) of the current state, this afternoon in preperation for doing this.... Im not sure how easy it is to tell default java from specified, as it just shows a path.. lets see if the updated list offers any insight |
I'm somewhat in two minds about whether to change the default or override in jenkins (and if we override we need to bear in mind me raising #2912 recently!) But I think an override for the remaining ones, then possibly look at adjusting the defaults and resetting the override after 2912 goes in is probably my preferred approach ... |
+1 from me on overriding. From what ive seen, the default is usually 8, so unless you guys have an objection I see no reason as to why we shouldnt override it with 17 (on platforms that have 17) |
@steelhead31 I guess the table from #2879 (comment) would suggest that the ones with an "empty" |
New audit sheet is available here.. |
I believe so for the online ones at least... they will be using the system default, or they may have been connected from the node itself with a different JDK from the default... I'll audit a few of them, and see if its obvious to discern those two cases. |
Looks like there's plenty of precedent and no objections to tweaking the jenkins agent config so I suggest we go for that, using |
@sxa I'm guessing build-azure-win2012r2-x64-4-sxa is on your home network? I cant get into it Since I deleted the jenkins service on build-alibaba-win2012r2-x64-1 and I cant get it back up, for now I have the jenkins agent running in a background process with java17. Steps to change the java path for the jenkins service:
|
No - the only ones hosted be me are the |
Actions arising from today's activities:
Follow-on actions: Change all the definitions once we implement #2912 ;-) |
Based on the new column from the plugin in #2950 there are five machines still running JDK8 for the agent: |
The final machines have all been upgraded to JDK17 for jenkins agents. |
Delete as appropriate from this list:
Details:
At the end of June Jenkins announced in a blog post that the Jenkins 2.357 and the forthcoming 2.361.1 LTS would require Java 11 or 17 on the server side (We are now running ours with Temurin 17).
With those new versions, using Java 8 for the jenkins agent systems will NOT BE SUPPORTED.
For this reason, we need to upgrade the jenkins agents to all run java 11 (or later) before we perform the next Jenkins LTS upgrade. We are currently on 2.346.3 (previous LTS) and will need to look at upgrading to 2.36.1. But before that, we need to ensure all the jenkins agents are running a suitable version.
Solaris in particular only has Java 8 on it (Temurin does not produce a later one) but there are versions of JDK11 for Solaris available from:
Or we could try building our own JDK11, but I don't think our machine configurations supported that the last time I tried.
FYA @karianna and I'll tag @steelhead31 too since he likes playing with Solaris!
The text was updated successfully, but these errors were encountered: