Skip to content

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 crosbymichael@gmail.com

Ref: containerd#3554

Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
@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)

zhsj added a commit to zhsj/cri that referenced this pull request Apr 13, 2020
Since containerd/containerd#3728
The docker/distribution/reference package is copied into containerd core

Signed-off-by: Shengjing Zhu <i@zhsj.me>
zhsj added a commit to zhsj/cri that referenced this pull request Apr 15, 2020
Since containerd/containerd#3728
The docker/distribution/reference package is copied into containerd core

Signed-off-by: Shengjing Zhu <i@zhsj.me>
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