Skip to content
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: 🤖 add --no-cache (poetry) and --no-cache-dir (pip) #680

Merged
merged 4 commits into from
Jan 19, 2023

Conversation

severo
Copy link
Collaborator

@severo severo commented Jan 19, 2023

to reduce the size of the docker images

Thanks @XciD

to reduce the size of the docker images

Thanks @XciD
@severo
Copy link
Collaborator Author

severo commented Jan 19, 2023

The compressed size of the worker's docker image has fallen from 7.91 GB to 5.45 GB!

Capture d’écran 2023-01-19 à 11 51 51

we don't need the GPU bindings since the server has no GPU. It should
reduce a lot the size of the docker image and the CI duration. Thanks
@XciD and @anton-l.
@codecov-commenter
Copy link

codecov-commenter commented Jan 19, 2023

Codecov Report

Base: 90.07% // Head: 90.07% // No change to project coverage 👍

Coverage data is based on head (60a0b7b) compared to base (37f53b1).
Patch has no changes to coverable lines.

❗ Current head 60a0b7b differs from pull request most recent head d1be77b. Consider uploading reports for the commit d1be77b to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #680   +/-   ##
=======================================
  Coverage   90.07%   90.07%           
=======================================
  Files          92       92           
  Lines        4574     4574           
=======================================
  Hits         4120     4120           
  Misses        454      454           
Flag Coverage Δ
jobs_mongodb_migration 84.33% <ø> (ø)
libs_libcommon 91.01% <ø> (ø)
services_admin 88.85% <ø> (ø)
services_api 89.87% <ø> (ø)
workers_datasets_based 91.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@severo severo merged commit 4d5f20b into main Jan 19, 2023
@severo severo deleted the reduce-docker-size branch January 19, 2023 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants