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

ansible: use Java 17 on AIX #2772

Closed
wants to merge 1 commit into from
Closed

ansible: use Java 17 on AIX #2772

wants to merge 1 commit into from

Conversation

richardlau
Copy link
Member

@richardlau richardlau commented Sep 24, 2021

Update the JRE used to run the Jenkins agent on AIX to Java 1117.
Also adds /opt/freeware/bin to the PATH for the agent to
pick up binaries from yum packages that no longer create symbolic
links in /usr/bin.

Refs: #2718

Trying this out on test-osuosl-aix72-ppc64_be-3 to see if we still hit issues with agent disconnects and/or zombie processes when the Jenkins agent is run with Java 1117.

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@sxa sxa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason to add /opt/freeware/bin to the START of the PATH? In general I'd consider it preferable to not override the default system utilities from /usr/bin where possible but there may be a good reason for doing it in this case so I won't block this if the comment can be dismissed :-)

ansible/roles/jenkins-worker/templates/aix.rc2.j2 Outdated Show resolved Hide resolved
@richardlau
Copy link
Member Author

Is there a reason to add /opt/freeware/bin to the START of the PATH? In general I'd consider it preferable to not override the default system utilities from /usr/bin where possible but there may be a good reason for doing it in this case so I won't block this if the comment can be dismissed :-)

It might not need to be at the start of the PATH. I'll try moving it to the end.

@sxa
Copy link
Member

sxa commented Oct 1, 2021

Maybe it should be java 17 now ;-)

@richardlau
Copy link
Member Author

Maybe it should be java 17 now ;-)

Tried. Can't find a Java 17 JRE for AIX via the api.adoptopenjdk.net rest API we're using in the Ansible scripts.

@sxa
Copy link
Member

sxa commented Oct 1, 2021

Maybe it should be java 17 now ;-)
Tried. Can't find a Java 17 JRE for AIX via the api.adoptopenjdk.net rest API we're using in the Ansible scripts.

That is a fair point. We haven't been able to release AIX17 formally yet although there are nightly builds that are available if you use ea instead of ga and go for hotpost instead of openj9 but they should obviously not necesarily be considered production ready. But they might be worth trying to see if they do show the same problems.

Update the JRE used to run the Jenkins agent on AIX to Java 17.
Also adds `/opt/freeware/bin` to the `PATH` for the agent to
pick up binaries from yum packages that no longer create symbolic
links in `/usr/bin`.
@richardlau
Copy link
Member Author

Updated to try Java 17 ea. (FWIW it appears that openj9 still worked 🤷.)

@richardlau richardlau changed the title ansible: use Java 11 on AIX ansible: use Java 17 on AIX Oct 1, 2021
@richardlau
Copy link
Member Author

Status on this is we're still encountering #2718 with Java 17 (and 11 before that) so just changing Java hasn't affected the Jenkins behavior. We may still want to use a later Java so I'll leave this open for now but we're focusing on trying to resolve #2718.

@richardlau
Copy link
Member Author

Superseded by #3047.

@richardlau richardlau closed this Oct 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants