-
Notifications
You must be signed in to change notification settings - Fork 8
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
Investigate caching buildkit cache volume #5
Comments
This would be nice to have. Long-term, I'd like to move over to one of Earthly's paid CI features (Satellites or Earthly CI), but I can't do that until I've convinced my team how great Earthly is :) |
I've made a little proof-of-concept for this using I think it works great! |
That looks great!! Any idea how this interacts with GitHub's 10GB per-repo cache limit? If the volume is >10GB, will the entire cache be evicted, or just individual layers/files |
@shepherdjerred I'm pretty sure the entire cache would be evicted. However, I think that's okay, it's just a cache after all. I don't mind clearing the cache and re-running all steps every once in a while. Now if your |
this looks cool! I wonder if |
The cache size is 10GB per repo, not workflow :/ Unfortunately, I think this would lead to a lot of evictions making the cache not very effective. The evictions might be tolerable if it's per layer, but if the entire volume is evicted then I think it'll just be wasted bandwidth, at least in my case. Great work though! I'm sure this would be useful for many others. |
I've no idea how the |
After @kevinschoonover added caching in #2 maybe it would be possible to cache in GHA whole buildkit
cache
volume.That should give huge speed boost cause it would work more/less as perfect case of advanced docker caching in earthly.
And speed of GH cache should be really good.
The text was updated successfully, but these errors were encountered: