-
Notifications
You must be signed in to change notification settings - Fork 93
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
Updating java openjdk to version 17 on rhel and debian builders #2293
Conversation
ec4773a
to
2ca6d5a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is tested on both distros? And we have an Ubuntu and a Red Hat Jenkins builder running jobs successfully after having this change applied?
Yes, It was ran against all of our upstream builders |
Resolve the merge conflict and I'll approve |
a5f883f
to
18cf775
Compare
Rebase back to 2ca6d5a and just remove the dashes in front of each var in builder.yml. I really don't want to merge a merge commit if we can avoid it. Ping me if you want help going over this on a call. |
5d98ff1
to
2ca6d5a
Compare
distros Signed-off-by: Adam Kraitman <[email protected]>
2ca6d5a
to
0dc684d
Compare
Updating java openjdk to version 17 on rhel and debian builders, I have added the update-alternatives command to make sure we set the OS to use the correct java version