-
Notifications
You must be signed in to change notification settings - Fork 317
Use a distroless based runtime image #18039
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
Conversation
5b9f7d9
to
7e5cbc6
Compare
ea7a677
to
d02f0cf
Compare
97fd25c
to
82c9a2a
Compare
82c9a2a
to
47e4c6e
Compare
done | ||
|
||
|
||
### | ||
### Stage 3: runtime | ||
### | ||
|
||
FROM docker.io/library/python:${PYTHON_VERSION}-slim-${DEBIAN_VERSION} | ||
FROM gcr.io/distroless/base-nossl-debian${DEBIAN_VERSION_NUMERIC}:debug |
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.
Is there any value in making the image unprivileged?
FROM gcr.io/distroless/base-nossl-debian${DEBIAN_VERSION_NUMERIC}:debug | |
FROM gcr.io/distroless/base-nossl-debian${DEBIAN_VERSION_NUMERIC}:debug-nonroot |
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.
Looping back on this, root is indeed necessary since the start script may take administrative actions like changing file owners and running as other users.
Depends on #18038
Supersedes #18033