Skip to content
This repository has been archived by the owner on Nov 9, 2024. It is now read-only.

Update docker base image from openjdk:8-jdk-alpine to openjdk:11-jdk-slim #48

Merged
merged 2 commits into from
May 8, 2020

Conversation

takami228
Copy link
Contributor

Update docker base image from openjdk-8 to openjdk-11.

There is no openjdk11 alpine image, only support 8 and 13 so it would be debian slim image. ( docker-library/openjdk#211 )

@takami228 takami228 changed the title Update docker base image from openjdk:8-alpine to openjdk:11-jdk-slim Update docker base image from openjdk:8-jdk-alpine to openjdk:11-jdk-slim Mar 20, 2020
@int128 int128 self-requested a review March 21, 2020 07:53
Copy link
Owner

@int128 int128 left a comment

Choose a reason for hiding this comment

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

Thank you for contribution. Would you check my comments?

Dockerfile Outdated
ENTRYPOINT java -jar httpstub.jar
Copy link
Owner

Choose a reason for hiding this comment

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

Use an array syntax here like ENTRYPOINT ["java", "-jar"]. See the section 4 of http://crosbymichael.com/dockerfile-best-practices.html.

Copy link
Owner

@int128 int128 left a comment

Choose a reason for hiding this comment

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

LGTM

@int128 int128 merged commit 842cec9 into int128:master May 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants