liferoad commented on code in PR #29001:
URL: https://github.com/apache/beam/pull/29001#discussion_r1384956414
##########
sdks/python/container/Dockerfile:
##########
@@ -82,6 +78,32 @@ RUN \
# Remove pip cache.
rm -rf /root/.cache/pip
+FROM gcr.io/distroless/cc-debian12 as beam
+LABEL Author "Apache Beam <[email protected]>"
+ARG TARGETOS
+ARG TARGETARCH
+
+# copy commands & libs to distroless
+COPY --from=python-base /bin /bin
Review Comment:
I do not think we need `/bin`. I added it here only for the convenience.
@tvalentyn I doubt users care about `apt` since if they need to touch this
file, they usually build their own images. The purpose for us to adopt
distorless is only to reduce the vulnerabilities when users use our containers
as the default.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
