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

Docker build cache might be in the wrong order #5186

Closed
teor2345 opened this issue Sep 19, 2022 · 0 comments · Fixed by #5190
Closed

Docker build cache might be in the wrong order #5186

teor2345 opened this issue Sep 19, 2022 · 0 comments · Fixed by #5190
Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles C-bug Category: This is a bug C-enhancement Category: This is an improvement I-cost Zebra infrastructure costs I-slow Problems with performance or responsiveness

Comments

@teor2345
Copy link
Contributor

Motivation

The branch cache is likely to be faster and cheaper than the main cache, because the code is more similar:

# We use multiple cache sources to confirm a cache hit, starting from the `main` branch cache,
# and if there's no hit, then continue with a cache scoped per branch.
cache-from: |
type=registry,ref=us-docker.pkg.dev/zealous-zebra/zebra/${{ inputs.image_name }}:main-cache
type=registry,ref=us-docker.pkg.dev/zealous-zebra/zebra/${{ inputs.image_name }}:${{ env.GITHUB_REF_SLUG_URL }}-cache

This is a medium priority, because we're trying to reduce costs.

@teor2345 teor2345 added C-bug Category: This is a bug A-devops Area: Pipelines, CI/CD and Dockerfiles C-enhancement Category: This is an improvement S-needs-triage Status: A bug report needs triage P-Medium ⚡ I-heavy Problems with excessive memory, disk, or CPU usage I-slow Problems with performance or responsiveness I-cost Zebra infrastructure costs and removed I-heavy Problems with excessive memory, disk, or CPU usage labels Sep 19, 2022
@mpguerra mpguerra moved this to 🆕 New in Zebra Sep 22, 2022
@mpguerra mpguerra added this to Zebra Sep 22, 2022
@mergify mergify bot closed this as completed in #5190 Sep 27, 2022
Repository owner moved this from 🆕 New to ✅ Done in Zebra Sep 27, 2022
@mpguerra mpguerra removed the S-needs-triage Status: A bug report needs triage label Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles C-bug Category: This is a bug C-enhancement Category: This is an improvement I-cost Zebra infrastructure costs I-slow Problems with performance or responsiveness
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants