-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add OCI artifact layer cache #7837
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #7837 +/- ##
==========================================
+ Coverage 48.72% 48.88% +0.15%
==========================================
Files 151 151
Lines 16265 16339 +74
==========================================
+ Hits 7925 7987 +62
- Misses 7377 7388 +11
- Partials 963 964 +1 |
08109d4
to
c636d26
Compare
/unhold ready for review. |
c636d26
to
d9a1a38
Compare
43fee55
to
29e5f09
Compare
78cd222
to
998b7c1
Compare
Added a cache max age. @cri-o/cri-o-maintainers @kwilczynski PTAL again |
998b7c1
to
8b793d3
Compare
We now utilize an on-disk key/value cache for OCI artifacts. The digests will be used to identify if a layer has to be re-pulled or not. Signed-off-by: Sascha Grunert <[email protected]>
8b793d3
to
863e514
Compare
/override ci/prow/ci-cgroupv2-integration |
@saschagrunert: Overrode contexts on behalf of saschagrunert: ci/prow/ci-cgroupv2-integration In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/retest |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kwilczynski, saschagrunert The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/remove-label release-note-none |
@kwilczynski: The label(s) In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@saschagrunert, not sure why bot does not seem to do what I ask. 🙂 If you could help me out here. Thank you! |
Done 👍 |
/retest-required |
What type of PR is this?
/kind feature
What this PR does / why we need it:
We now utilize an on-disk key/value cache for OCI artifacts. The digests will be used to identify if a layer has to be re-pulled or not. The artifact API now returns a path to the file on disk.
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?