-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
vendor: update docker/docker master, and remove uses of github.com/moby/sys/user #5708
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5708 +/- ##
==========================================
- Coverage 59.54% 59.13% -0.42%
==========================================
Files 346 343 -3
Lines 29381 29370 -11
==========================================
- Hits 17496 17369 -127
- Misses 10914 11029 +115
- Partials 971 972 +1 |
a5ec623
to
cf6bebd
Compare
removes dependency on github.com/moby/sys/user Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
cf6bebd
to
ee8ad8a
Compare
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
After this, pkg/pools is still in use in the CLI for the Classic Builder; cli/cli/command/image/build/context.go Lines 426 to 447 in 3b49deb
That was;
However, on that PR, it was suggested to move the |
Failures look legit / related;
|
testing changes from Split idtools to an internal package and package to be moved moby/moby#49087pkg/pools
in archive moby/moby#49117- What I did
- How I did it
- How to verify it
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)