Skip to content

Conversation

guillaumerose
Copy link
Contributor

This code is already used by Hub and helps us GC old content.

Manish Tomar added 3 commits December 5, 2019 20:48
Signed-off-by: Guillaume Rose <guillaume.rose@docker.com>
Signed-off-by: Guillaume Rose <guillaume.rose@docker.com>
Signed-off-by: Guillaume Rose <guillaume.rose@docker.com>
@codecov
Copy link

codecov bot commented Dec 5, 2019

Codecov Report

Merging #3061 into master will increase coverage by <.01%.
The diff coverage is 75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3061      +/-   ##
==========================================
+ Coverage   60.73%   60.74%   +<.01%     
==========================================
  Files         102      102              
  Lines        8073     8077       +4     
==========================================
+ Hits         4903     4906       +3     
- Misses       2518     2519       +1     
  Partials      652      652
Flag Coverage Δ
#linux 60.74% <75%> (ø) ⬆️
Impacted Files Coverage Δ
registry/storage/registry.go 88.34% <100%> (+0.07%) ⬆️
registry/storage/paths.go 70.54% <66.66%> (-0.1%) ⬇️

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 dee21c0...c9c3324. Read the comment docs.

@manishtomar
Copy link
Contributor

@guillaumerose I wonder if we can add tests for blobs.Enumerate. Thanks a lot for doing this :). Ping @dmcgowan @caervs Kindly review.

Signed-off-by: Guillaume Rose <guillaume.rose@docker.com>
@guillaumerose
Copy link
Contributor Author

I added 2 extra unit tests. It should be better now.

Copy link
Contributor

@manishtomar manishtomar left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

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

LGTM

@dmcgowan dmcgowan merged commit f187812 into distribution:master Dec 6, 2019
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.

4 participants