-
Notifications
You must be signed in to change notification settings - Fork 141
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
Improve : Optimize Dockerfile for Better Build Efficiency #126
base: main
Are you sure you want to change the base?
Improve : Optimize Dockerfile for Better Build Efficiency #126
Conversation
This branch name and pull request heading are clear and descriptive. The branch name indicates that the pull request is related to a feature, specifically the optimization of the Dockerfile. The heading of the pull request provides a concise summary of the changes made in the pull request.
@ruomingp Hey, could you please help me with this check? How do I pass this check? |
It seems a bug in the Dockerfile?
|
RUN echo "deb [signed-by=/usr/share/keyrings/cloud.google.gpg] http://packages.cloud.google.com/apt cloud-sdk main" | tee -a /etc/apt/sources.list.d/google-cloud-sdk.list && \ | ||
# Install necessary packages in a single layer. | ||
RUN apt-get update && \ | ||
apt-get install -y apt-transport-https ca-certificates gnupg curl gcc g++ git jq screen ca-certificates && \ |
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.
Seems like some of these are repeated from above (L10-11)?
Dockerfile
Outdated
@@ -54,24 +46,31 @@ RUN ./run_tests.sh "${PYTEST_FILES}" | |||
# Bastion container spec. # | |||
################################################################################ | |||
|
|||
# Bastion container for application deployment. |
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.
nit: These comments are already covered in the comment blocks for each build target, e.g. L46 or L59.
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.
Looks like the image is still failing to build. Have you tested building this image locally?
What does this PR do?
This pull request optimizes the Dockerfile by reducing the number of image layers, combining instructions, and improving readability. It also adds labels for better documentation. The Dockerfile is now more efficient and follows best practices for building Docker images.