-
Notifications
You must be signed in to change notification settings - Fork 32
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
Enable the fetching of only the OCI descriptor #2220
base: main
Are you sure you want to change the base?
Enable the fetching of only the OCI descriptor #2220
Conversation
dbc2510
to
ee8df44
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
ee8df44
to
5583a73
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2220 +/- ##
==========================================
+ Coverage 72.42% 72.69% +0.27%
==========================================
Files 88 88
Lines 7539 7615 +76
==========================================
+ Hits 5460 5536 +76
Misses 2079 2079
Flags with carried forward coverage won't be shown. Click here to find out more.
|
/ok-to-test |
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.
How do you feel about adding other properties to the returned object?
/ok-to-test |
Do a |
938e1f8
to
f2e2d19
Compare
Some OCI artifacts need to have a specific media type (i.e. they should only be an OCI image index or an OCI image manifest). In order to determine that with a single remote call, we can just get the descriptor and then check the mediaType of that object. Signed-off-by: arewm <arewm@users.noreply.github.com>
f2e2d19
to
88ce5a8
Compare
Thanks. Squashed and regenerated the docs. |
/ok-to-test |
Some OCI artifacts need to have a specific media type (i.e. they should only be an OCI image index or an OCI image manifest). In order to determine that with a single remote call, we can just get the descriptor and then check the mediaType of that object.