Skip to content

Add distribution subpkgs to core#3728

Merged
estesp merged 1 commit intocontainerd:masterfrom
crosbymichael:dist-rm
Oct 7, 2019
Merged

Add distribution subpkgs to core#3728
estesp merged 1 commit intocontainerd:masterfrom
crosbymichael:dist-rm

Conversation

@crosbymichael
Copy link
Member

@crosbymichael crosbymichael commented Oct 7, 2019

Ref: #3554

Ports over some of the distribution/* packages into the containerd to core so that we reduce that dependency.

Signed-off-by: Michael Crosby [email protected]

@jterry75
Copy link
Contributor

jterry75 commented Oct 7, 2019

Can we remove this from vendor now too?

@theopenlab-ci
Copy link

theopenlab-ci bot commented Oct 7, 2019

Build succeeded.

@crosbymichael
Copy link
Member Author

@jterry75 not yet, i'll have to merge this one and then update CRI

@codecov-io
Copy link

codecov-io commented Oct 7, 2019

Codecov Report

Merging #3728 into master will increase coverage by 0.03%.
The diff coverage is 45.23%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3728      +/-   ##
==========================================
+ Coverage    42.1%   42.13%   +0.03%     
==========================================
  Files         129      131       +2     
  Lines       14307    14474     +167     
==========================================
+ Hits         6024     6099      +75     
- Misses       7383     7467      +84     
- Partials      900      908       +8
Flag Coverage Δ
#linux 45.58% <43.47%> (-0.04%) ⬇️
#windows 37.16% <45.23%> (+0.1%) ⬆️
Impacted Files Coverage Δ
remotes/docker/fetcher.go 50% <100%> (ø) ⬆️
remotes/docker/errdesc.go 30.76% <30.76%> (ø)
remotes/docker/errcode.go 49.21% <49.21%> (ø)

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 8d0d1eb...901bcb2. Read the comment docs.

@jterry75
Copy link
Contributor

jterry75 commented Oct 7, 2019

@jterry75 not yet, i'll have to merge this one and then update CRI

Oh the stepchild CRI :)

Copy link
Contributor

@jterry75 jterry75 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
Member

@cpuguy83 cpuguy83 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
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

@thaJeztah
Copy link
Member

Note: looks like this needed vndr to be run again to remove the files that are no longer used (see #3759 (comment)) (in case this is something that's backported)

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.

6 participants