-
Notifications
You must be signed in to change notification settings - Fork 252
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
Containers image registry implementation with opt-in caching #1626
base: master
Are you sure you want to change the base?
Containers image registry implementation with opt-in caching #1626
Conversation
Skipping CI for Draft Pull Request. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: joelanford 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 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1626 +/- ##
==========================================
+ Coverage 46.77% 55.14% +8.36%
==========================================
Files 135 136 +1
Lines 15883 15911 +28
==========================================
+ Hits 7430 8774 +1344
+ Misses 7412 5984 -1428
- Partials 1041 1153 +112 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
a0e5296
to
a88e1e3
Compare
Signed-off-by: Joe Lanford <[email protected]>
Signed-off-by: Joe Lanford <[email protected]>
a88e1e3
to
07676d6
Compare
Signed-off-by: Joe Lanford <[email protected]>
07676d6
to
caec04f
Compare
Description of the change:
This change adds a new
containers/image
-based registry client and updates various (non-deprecated) commands to use it. Specificallyrender
(and other commands that useaction.Render
) andalpha bundle validate
. Deprecated sqlite based commands are not updated.The intent of the PR is to keep essentially full compatibility with the
containerd
implementation, and to additionally gain the features of containers/image-based clients.This PR also includes the ability to opt into caching (and reusing) downloaded blobs (when
OLM_CACHE_DIR
is set). The behavior around the cache is essentially--pull-policy=Always
, where existing layers do not need to be redownloaded. While this PR does garbage collect unused blobs in the cache after each image pull, specifically cleaning up the cache is (for now at least) left as an exercise for the user.Motivation for the change:
Improves alignment of
opm
with the ecosystem of tools it is often used with. The main benefit here is thatopm
calls that use this client will now honor user's existingcontainers/image
-based configuration, includingregistries.conf
CAs and image mappings.Reviewer Checklist
/docs