-
Notifications
You must be signed in to change notification settings - Fork 3.7k
Add distribution subpkgs to core #3728
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
Conversation
Ref: containerd#3554 Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
Can we remove this from vendor now too? |
Build succeeded.
|
@jterry75 not yet, i'll have to merge this one and then update CRI |
Codecov Report
@@ 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
Continue to review full report at Codecov.
|
Oh the stepchild CRI :) |
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
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
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
Note: looks like this needed |
Since containerd/containerd#3728 The docker/distribution/reference package is copied into containerd core Signed-off-by: Shengjing Zhu <i@zhsj.me>
Since containerd/containerd#3728 The docker/distribution/reference package is copied into containerd core Signed-off-by: Shengjing Zhu <i@zhsj.me>
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