-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Dockerfile/chore: Update Dockerfile #5097
Conversation
…sed image size by optimization.
Wait, so you reduced the Docker image size by a few MB and then also increased it by 10 MB with And wouldn't it be even better to cram all shell commands into a single RUN block?
edit: added |
Try So generally cramming all the dependencies or packages is good. But joining the So joining the pip package is efficient and syntactically looks good. Let's install over 100 packages without any CMD, ENTRYPOINT. example Dockerfile: over 10 minutes. Also if one of them is updated you need to spend a lot of time, again.
example Dockerfile2 : almost 1.5 minutes. Even one of them is updated still almost 1.5 minutes
|
Reduce a layer, update pip and apk, and reduce increased image size by optimization.
The previous image size is approximately 247 MB. With this PR, 244 MB
Without the code below
the image will be 10MB smaller.