bvolpato commented on code in PR #29001:
URL: https://github.com/apache/beam/pull/29001#discussion_r1384310104
##########
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:
Not a blocker here, just a general comment: I'd rather pick specific
binaries rather than copying every `/bin` binary. I understand it probably
means more breakages, but it's also a smaller vulnerability surface.
--
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]
