-
Notifications
You must be signed in to change notification settings - Fork 26
github: fix caching the docker images #176
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #176 +/- ##
=======================================
Coverage 76.25% 76.26%
=======================================
Files 130 130
Lines 33970 33970
=======================================
+ Hits 25904 25906 +2
+ Misses 8066 8064 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
1c44e67
to
e8326b5
Compare
e8326b5
to
7683a03
Compare
Expanded the commit message a bit:
1: actions/cache#1452 Also removed the I am especially interested in seeing @smeso review on this PR because you created the original caching approach. |
144c5a2
to
456a5ae
Compare
This commit updates the way docker cache is created by including the current date into the cache key. Now when "save-always" option in actions/cache is deprecated [1] this seems to be the conventional way [2] to create the cache. [1] actions/cache#1452 [2] https://github.com/actions/cache?tab=readme-ov-file#creating-a-cache-key Signed-off-by: Mikhail Koviazin <mikhail.koviazin@aiven.io>
456a5ae
to
6c97033
Compare
Hey @bogdanp05 , I've resolved our conversation. Your approach worked, thanks! Please approve if it looks good to you, it'll auto-merge. |
This PR supercedes #170 and fixes #140
Pull Request check-list
Description of change