Skip to content

Conversation

@dmcgowan
Copy link
Member

@dmcgowan dmcgowan commented Jul 9, 2019

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

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why ErrNotFound?

Copy link
Member Author

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.

@dmcgowan dmcgowan force-pushed the registry-configuration branch 2 times, most recently from 06ce917 to 22625f4 Compare July 11, 2019 17:37
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jul 11, 2019

Build succeeded.

@containerd containerd deleted a comment from theopenlab-ci bot Jul 11, 2019
@containerd containerd deleted a comment from theopenlab-ci bot Jul 11, 2019
@codecov-io
Copy link

codecov-io commented Jul 11, 2019

Codecov Report

Merging #3400 into master will increase coverage by 0.01%.
The diff coverage is 51%.

Impacted file tree graph

@@            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
Flag Coverage Δ
#linux 48.96% <51.21%> (-0.03%) ⬇️
#windows 40.36% <51%> (+0.09%) ⬆️
Impacted Files Coverage Δ
remotes/docker/pusher.go 0% <0%> (ø) ⬆️
remotes/docker/fetcher.go 50% <40.67%> (-14.59%) ⬇️
remotes/docker/registry.go 60.27% <60.27%> (ø)
remotes/docker/resolver.go 54.26% <61.86%> (-1.74%) ⬇️
remotes/docker/authorizer.go 70.52% <77.14%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d6be45e...0b29c9c. Read the comment docs.

@dmcgowan dmcgowan force-pushed the registry-configuration branch from 22625f4 to 06582cb Compare July 12, 2019 22:39
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jul 12, 2019

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>
@dmcgowan dmcgowan force-pushed the registry-configuration branch from 06582cb to 0b29c9c Compare July 16, 2019 18:28
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jul 16, 2019

Build succeeded.

Copy link
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@crosbymichael
Copy link
Member

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants