-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Update resolver to handle endpoint configuration #3400
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
Update resolver to handle endpoint configuration #3400
Conversation
remotes/docker/resolver.go
Outdated
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 ErrNotFound?
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.
The end result to the caller is the same, no manifest could be returned. Whether that is because it does not exist or it could not find a valid manifest won't make a difference.
06ce917 to
22625f4
Compare
|
Build succeeded.
|
Codecov Report
@@ Coverage Diff @@
## master #3400 +/- ##
==========================================
+ Coverage 45.05% 45.06% +0.01%
==========================================
Files 113 114 +1
Lines 12563 12739 +176
==========================================
+ Hits 5660 5741 +81
- Misses 6049 6131 +82
- Partials 854 867 +13
Continue to review full report at Codecov.
|
22625f4 to
06582cb
Compare
|
Build succeeded.
|
Adds support for registry mirrors Adds support for multiple pull endpoints Adds capabilities to limit trust in public mirrors Fixes user agent header missing Signed-off-by: Derek McGowan <derek@mcgstyle.net>
06582cb to
0b29c9c
Compare
|
Build succeeded.
|
estesp
left a comment
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
|
LGTM |
Adds support for registry mirrors
Adds support for multiple pull endpoints
Adds registry capabilities to limit trust in public mirrors or avoid pushing to mirrors
Adds support for generic header addition