Skip to content

Update base image version in multi-stage Dockerfile build - #23911

Merged
dvdksn merged 1 commit into
docker:mainfrom
ssmf:patch-1
Jul 24, 2026
Merged

dvdksn merged 1 commit into
docker:mainfrom
ssmf:patch-1

Conversation

@ssmf

@ssmf ssmf commented Jan 2, 2026 •

Copy link
Copy Markdown
Contributor

Description

the example's original Dockerfile did not reflect the docs content:

The second stage is a new stage named final. It uses a slimmer eclipse-temurin:21.0.2_13-jre-jammy image

I also verified that using this image indeed works properly with the provided example

@netlify

netlify Bot commented Jan 2, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 6f8977d
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/696174d9b90fbb0008fe219f
😎 Deploy Preview https://deploy-preview-23911--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the area/get-started Relates to get started and onboarding docs label Jan 2, 2026
RUN ./mvnw clean install

FROM eclipse-temurin:21.0.8_9-jre-jammy AS final
FROM eclipse-temurin:21.0.2_13-jre-jammy AS final

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like the version was bumped in #23361, but that we forgot to update the description below. Could you please change the description instead to reflect the newer version? 🙏

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done 👍

the previous version was not reflecting the docs contents
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the PR. We'd like to make our product docs better, but haven't been able to review all the suggestions. As our docs change often and quickly diverge, we do not have the bandwidth to review and rebase old PRs.

If the updates are still relevant, please rebase your PR against the latest version of the docs and add a comment when it's ready. This helps our maintainers focus on active contributions. If there's no activity, this PR will be closed in 30 days.

To remove the stale label: Comment /lifecycle active
To freeze (requires write access): Comment /lifecycle frozen

@dvdksn dvdksn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks. Not sure what took me so long to get this across the line. sorry 😅

@dvdksn
dvdksn merged commit 4060581 into docker:main Jul 24, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/get-started Relates to get started and onboarding docs lifecycle/stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants