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

Add java 17 preview #201

Merged
merged 2 commits into from
Sep 26, 2021
Merged

Add java 17 preview #201

merged 2 commits into from
Sep 26, 2021

Conversation

timja
Copy link
Member

@timja timja commented Sep 23, 2021

Build will fail till docker-library/official-images#10945 is merged

@timja timja requested a review from a team as a code owner September 23, 2021 17:28
Ubuntu definitely has binutils installed in the image =/
Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

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

Approved but with a question on the change to a less precise debian image version

@@ -33,7 +33,7 @@ RUN jlink \
--compress=2 \
--output /javaruntime

FROM debian:bullseye-20210816
FROM debian:bullseye
Copy link
Contributor

Choose a reason for hiding this comment

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

I prefer the date stamp on the Docker image so that we track how close we are to the current date. Is there a compelling reason why this needs to remove the date code?

Suggested change
FROM debian:bullseye
FROM debian:bullseye-20210816

Copy link
Member Author

Choose a reason for hiding this comment

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

dependabot doesn't work with it and we don't have enough maintainers.
We have a higher chance of staying up to date if we don't fix it to this tag.

but if you feel strongly I can change back

Copy link
Member Author

Choose a reason for hiding this comment

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

can do a follow up if required

@timja timja merged commit 5bc3842 into jenkinsci:master Sep 26, 2021
@timja timja deleted the add-java-17-preview branch September 26, 2021 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants