-
Notifications
You must be signed in to change notification settings - Fork 27
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
Use temurin 8 docker image #120
Conversation
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.
We should look at upgrading to at least Java 11 at some point.
Every project, that invokes the Jenkins Jira API is a nightmare to upgrade 🥴 |
Ha yup I've hit that before, generally you can drop that dep and use the client directly but the atlassian client isn't great. |
Yeah, also used for the RPU, but hey, after my changes today, the RPU's minimum Java version is now Java 11 👏🏻 |
Requested your review, slide's doesn't suffice to merge this pr, according to GH. |
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.
Good idea !
We must stick to a JRE (as it is a production image)
Co-authored-by: Damien Duportal <damien.duportal@gmail.com>
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.
nice job!
I can't locate the current image on the docker hub, the change proposed uses temurin builds.