-
Notifications
You must be signed in to change notification settings - Fork 518
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 remote.Puller #1644
Add remote.Puller #1644
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1644 +/- ##
==========================================
- Coverage 72.07% 72.03% -0.04%
==========================================
Files 120 121 +1
Lines 9772 9860 +88
==========================================
+ Hits 7043 7103 +60
- Misses 2010 2033 +23
- Partials 719 724 +5
... and 1 file with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
f9faf99
to
a1568b4
Compare
@@ -25,8 +25,9 @@ import ( | |||
"github.com/google/go-containerregistry/pkg/v1/remote/transport" | |||
) | |||
|
|||
type catalog struct { | |||
type Catalogs struct { |
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.
Why plural?
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.
remote.Catalog
is already defined 😅
pkg/v1/remote/descriptor.go
Outdated
func (f *fetcher) head(ctx context.Context, ref name.Reference) (*v1.Descriptor, error) { | ||
return f.headManifest(ctx, ref, allManifestMediaTypes) | ||
} |
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.
This method doesn't add much, can we just have callers pass allManifestMediaTypes
themselves?
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.
Done.
This PR adds a Puller implementation and uses it in `crane ls` and `crane catalog` to stream results for each page.
a1568b4
to
8d104df
Compare
This PR adds a Puller implementation and uses it in
crane ls
andcrane catalog
to stream results for each page.