-
-
Notifications
You must be signed in to change notification settings - Fork 232
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 agents with Jdk17 error: hudson.remoting.ProxyException: java.lang.reflect.InaccessibleObjectException: Unable to make field private final java.util.Comparator java.util.TreeMap.comparator accessible: module java.base does not "opens java.util" to unnamed module @1c1618cf #207
Comments
Sure it'll need something like: I had that initially but then I removed it as I wasn't able to trigger any of these issues. Feel like contributing a fix? |
Hey @timja, thanks for reaching back, you mean to try adding those flags to this entry point https://github.com/jenkinsci/docker-inbound-agent/blob/master/jenkins-agent and test the agent again if works create a PR adding those flags for jdk17 entry point? ( https://github.com/jenkinsci/docker-inbound-agent/blob/master/jenkins-agent) |
Yes, bearing in mind that the entrypoint here uses |
Thanks @timja Connection Command
Error log
|
According to the error you need to add java.lang.reflect to the modules list |
Hey @timja I have added the following OPTS I still got an error @timja you mentioned you had this initially but then removed later, did you manage to fix error without these |
Next error is java.text initially I copied the options from jenkinsci/docker but then I tested without and found that basic steps worked fine without it |
Ya just added the following, running some tests
|
What made you add java.awt.font? |
just removed the java.awt.font, not including in my tests, added it as I copy the --add-opens from the existing app. |
yes it requires --add-opens java.desktop/java.awt.font=ALL-UNNAMED
|
You got a bigger stacktrace for it? |
Hey @timja sorry for late reply, below I have shared the error log
|
Interesting, we can likely work around that in Jenkins core by removing that converter which we don't need, but it can be added to the |
True So moving forward @timja for users to use jdk17 should I add a PR for https://github.com/jenkinsci/docker-inbound-agent/blob/master/jenkins-agent or will it be added in Jenkins core with the next version? |
Add a PR for here for now please |
@timja sure, here do you suggest to add it for jdk17 docker image or create a new entry point file? |
same way as in jenkinsci/docker entrypoint, set an environment variable in the dockerfile https://github.com/jenkinsci/docker/blob/master/17/debian/bullseye/hotspot/Dockerfile#L105 then https://github.com/jenkinsci/docker/blob/master/jenkins.sh#L38 (remembering that script is bash, while this is using |
Does anyone have steps to reproduce this issue from scratch? I was thinking about moving the |
I tried to reproduce this and wasn't able to. =/ @bob-rohan do you have an example you could share from here? jenkinsci/docker-inbound-agent#252 |
I don't @timja , I not longer have access for the client where this work was done. I'll reach out see if someone could pick this up |
So maybe we should conclude that these |
we could remove for now and see if someone can provide a way to reproduce if we hit it. |
One YOLO PR coming up. |
This comment was marked as off-topic.
This comment was marked as off-topic.
@shawngmc That stack trace seems to come from the controller, so I have no idea why you are posting it on an issue in the |
Argh - you're right. That is from the controller. I think I simply copied/pasted into the wrong GitHub tab; pretty sure I meant to paste into jenkinsci/docker-plugin#905 Getting Jenkins itself, the docker plugin, and a docker jenkins worker image all talking together happily is... painful. |
Ping on this issue: is it still reproduceable or shall we close it? |
This ticket was filed in the wrong repository and was fixed in jenkinsci/jenkins#7270. |
Version report
Using jenkins master: 2.249.2
Jenkins remoting version: 4.10
Connection Protocol: websocket
Java version: openjdk version "17" 2021-09-14
OpenJDK Runtime Environment Temurin-17+35 (build 17+35)
OpenJDK 64-Bit Server VM Temurin-17+35 (build 17+35, mixed mode)
Reproduction steps
WARNING: A terminally deprecated method in java.lang.System has been called
WARNING: System::setSecurityManager has been called by hudson.remoting.jnlp.Main (file:/usr/share/jenkins/agent.jar)
WARNING: Please consider reporting this to the maintainers of hudson.remoting.jnlp.Main
WARNING: System::setSecurityManager will be removed in a future release
The text was updated successfully, but these errors were encountered: