-
Notifications
You must be signed in to change notification settings - Fork 138
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
feat: oci directory pusher #1702
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lkingland 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 ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #1702 +/- ##
==========================================
- Coverage 62.64% 62.44% -0.21%
==========================================
Files 97 95 -2
Lines 12906 12383 -523
==========================================
- Hits 8085 7732 -353
+ Misses 4054 3950 -104
+ Partials 767 701 -66
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lol I see @matejvasek and I are making the same comments - I'll stop now.
/lgtm
/hold for requested changes
960a7cf
to
2aadd54
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/hold to remove debugging output
bff1617
to
8098117
Compare
8098117
to
003f0ac
Compare
0d2a25a
to
2f1ebac
Compare
2f1ebac
to
5466a90
Compare
5466a90
to
3e073da
Compare
/hold cancel |
/hold cancel |
@lkingland this is still marked as a work in progress - is it ready to go? |
Yes, I am trying a slightly new process. To reduce complexity and thus my cognitive load, I am structuring these Scaffolding-related PRs as being strictly serially processed, presuming a latter PR depends on the one prior in all cases (even when in theory it could be merged concurrently). Since the prior PR was merged, this one could be taken out of draft and merged as well, but this way its preserving the order. So far this small capitulation has led to an appreciable improvement in my workflow |
For use by the initial implementation of Scaffolding, this is a basic container pusher which sends a directory which is formatted according to the OCI standard to a given registry using the default keychain, insecurely, or using explicit creds.
/kind enhancement