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

Enable poetry cache in CI #15

Closed
wants to merge 12 commits into from
Closed

Conversation

andreacamisa
Copy link
Owner

@andreacamisa andreacamisa commented Aug 5, 2023

When docker layers are not found in the cache, poetry install gets executed again, downloading all packages each time. Making sure that poetry looks up packages in the cache would speed up re-building of those layers.

@andreacamisa andreacamisa self-assigned this Aug 5, 2023
@andreacamisa andreacamisa force-pushed the acamisa/poetry-cache branch from 7cf11b6 to 35f50ed Compare August 5, 2023 11:50
@andreacamisa
Copy link
Owner Author

andreacamisa commented Aug 5, 2023

Tried to reproduce approach explained here but cache mounts with "from" argument are not actually bound to the specified directories. (Also, cache name with hash of poetry lock was not computed correctly)

Will need to wait until the cache mounts are implemented: moby/buildkit#1512
Workaround (but it's not worth it right now): docker/buildx#244 (comment)

Further references:

@andreacamisa andreacamisa deleted the acamisa/poetry-cache branch August 5, 2023 12:51
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.

1 participant