-
Notifications
You must be signed in to change notification settings - Fork 7
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
Upgrade docker base image #49
Conversation
@@ -22,5 +22,5 @@ RUN useradd --create-home app \ | |||
|
|||
USER app | |||
|
|||
ENTRYPOINT ["/usr/bin/java"] | |||
ENTRYPOINT ["/opt/java/openjdk/bin/java"] |
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.
❯ docker run -it eclipse-temurin:17 /bin/bash
root@d2eec3087d0c:/# which java
/opt/java/openjdk/bin/java
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.
Upgrade 対応ありがとうございます!
変更内容自体は問題なさそうに見えています 🙆♂️
ちなみに動作確認としては何が確認できればOKと言えそうですかね...?
レビューありがとうございます!
docker build出来ると良さそうかなと思います。が、少し手間がかかります😖
dockerfileを以下のように書き換える
感じですかね 👀 |
手順ありがとうございます!
ここでエラーが出たので、確認中です 🙏
|
確かに /src/main 以下には kotlin というフォルダしか無いので、java ってフォルダは無いなとなっている |
手元の環境で |
このエラーメッセージでググってみました。
|
@kuroponzu こちらでした〜!
|
path を通したら成功するようになった 👏
|
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.
docker build .
が成功することが確認できました!LGTM!
summary
Upgrade the docker base image.
Why change it?
Bump dependencies
Bump dependencies and use java 17.
#48
Base image is deprecated
openjdk_8 is now deprecated.
https://hub.docker.com/_/openjdk/
details
I choise the images of eclipse-temurin, because it not depends on identification platform.
ex) amazoncorretto is for aws-service
However, I don't have any strong opinion about it, so if you have any images to recommend, let me know.
ref) https://future-architect.github.io/articles/20211220a/