-
Notifications
You must be signed in to change notification settings - Fork 50
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
Reduce docker image size #394
Conversation
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.
So cool! Thanks a lot @sanders41 for this PR
LGTM! 🔥
bors merge |
This message is sent automatically Thank you very much for submitting your PR! Did you know that throughout the month of June we’re holding a rafle? |
@@ -5,9 +5,15 @@ ENV LANG C.UTF-8 | |||
|
|||
WORKDIR /docs-scraper | |||
|
|||
RUN apt-get update -y && apt-get install -y python3-pip libnss3 chromium-driver | |||
RUN : \ |
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.
What is this :
notation in this line RUN : \
@sanders41? I never saw that!
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.
It just let me split it across multiple lines from the start.
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.
nice, cool!! :D
Pull Request
With these changes I got ~ a 43% reduction in the final image size (1.55GB to 877MB). Note that depending on your local files present (virtual environments, etc) you may see slightly different numbers with a build, but it will always be significantly smaller.
Related issue
Fixes #<issue_number>
What does this PR do?
apt-get install
section.dockerignore
filepip3
. Because the image ispython:3.10.8-slim-bullseye
pip
is already included by defaultPR checklist
Please check if your PR fulfills the following requirements:
Thank you so much for contributing to Meilisearch!