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
chore(dp): Remove apt packages version pinning in DP Dockerfiles #13769
Conversation
Signed-off-by: Tomasz Gromowski <tomasz@freedomfi.com>
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
@@ -1,7 +1,7 @@ | |||
ARG ENV=standard | |||
FROM python:3.9.2-slim-buster as protos-generator | |||
|
|||
RUN apt-get update && apt-get install -y --no-install-recommends curl=7.64.0-4+deb10u2 zip=3.0-11+b1 make=4.2.1-1.2 unzip=6.0-23+deb10u2 | |||
RUN apt-get update && apt-get install -y --no-install-recommends curl zip make unzip |
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.
@@ -1,7 +1,7 @@ | |||
ARG ENV=standard | |||
FROM python:3.9.2-slim-buster as protos-generator | |||
|
|||
RUN apt-get update && apt-get install -y --no-install-recommends curl=7.64.0-4+deb10u2 zip=3.0-11+b1 make=4.2.1-1.2 unzip=6.0-23+deb10u2 | |||
RUN apt-get update && apt-get install -y --no-install-recommends curl zip make unzip |
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.
@@ -1,7 +1,7 @@ | |||
ARG ENV=standard | |||
FROM python:3.9.2-slim-buster as protos-generator | |||
|
|||
RUN apt-get update && apt-get install -y --no-install-recommends curl=7.64.0-4+deb10u2 zip=3.0-11+b1 make=4.2.1-1.2 unzip=6.0-23+deb10u2 | |||
RUN apt-get update && apt-get install -y --no-install-recommends curl zip make unzip |
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.
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.
LGTM. Ignoring hadolint remarks, hard pinning versions is ok, but we can reply on distro packaging here. Versions are not pinned/specified but those are system packages that should not change much within base release of the image.
💚 All backports created successfully
Questions ?Please refer to the Backport tool documentation and see the Github Action logs for details |
) Signed-off-by: Tomasz Gromowski <tomasz@freedomfi.com> Signed-off-by: Tomasz Gromowski <tomasz@freedomfi.com> (cherry picked from commit 77260f2)
…ma#13769) Signed-off-by: Tomasz Gromowski <tomasz@freedomfi.com> Signed-off-by: Tomasz Gromowski <tomasz@freedomfi.com>
…ma#13769) Signed-off-by: Tomasz Gromowski <tomasz@freedomfi.com> Signed-off-by: Tomasz Gromowski <tomasz@freedomfi.com>
…ma#13769) (magma#13776) Signed-off-by: Tomasz Gromowski <tomasz@freedomfi.com> Signed-off-by: Tomasz Gromowski <tomasz@freedomfi.com> (cherry picked from commit 77260f2) Co-authored-by: Tomasz Gromowski <tomasz@freedomfi.com>
Signed-off-by: Tomasz Gromowski tomasz@freedomfi.com
Summary
To avoid dependency issues, when the base Docker image is updated, we shouldn't pin apt to a particular package version.
Test Plan
Additional Information