Skip to content

Bump moby vendor and dependencies - #892

Merged
vdemeester merged 6 commits into
docker:masterfrom
thaJeztah:bump-moby-vendor
Feb 20, 2018
Merged

Bump moby vendor and dependencies#892
vdemeester merged 6 commits into
docker:masterfrom
thaJeztah:bump-moby-vendor

Conversation

@thaJeztah

@thaJeztahthaJeztah commented Feb 20, 2018

Copy link
Copy Markdown
Member

- Add API support for SCTP port mapping
- Add canonical import path
- Add `REMOVE` and `ORPHANED` to TaskState
- Fix TLS from environment variables in client
- Introduce NewClientWithOpts func to build custom client easily
- Wrap response errors for container copy methodsto fix error detection using
`IsErrNotFound` and `IsErrNotImplemented` for `ContainerStatPath`,
`CopyFromContainer`, and `CopyToContainer` methods.
- Produce errors when empty ids are passed into inspect calls
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
- Support parsing SCTP port mapping
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

@vdemeestervdemeester left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 🦁

@cpuguy83

Copy link
Copy Markdown
Collaborator

Looks like CircleCI is having some problems starting (some of) the tests.

@cpuguy83

Copy link
Copy Markdown
Collaborator

I don't have access to trigger a rebuild in Circle.

@codecov-io

codecov-io commented Feb 20, 2018

Copy link
Copy Markdown

Codecov Report

Merging #892 into master will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@## master #892 +/- ##
======================================
Coverage 53.2% 53.2% ======================================
Files 258 258 Lines 16338 16338 ======================================
Hits 8693 8693 Misses 7080 7080 Partials 565 565

@dnephindnephin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@vdemeester
vdemeester merged commit 02e8dc6 into docker:masterFeb 20, 2018
@GordonTheTurtleGordonTheTurtle added this to the 18.03.0 milestone Feb 20, 2018
@thaJeztah
thaJeztah deleted the bump-moby-vendor branch February 20, 2018 17:55
nobiit pushed a commit to nobidev/docker-cli that referenced this pull request Nov 19, 2025
Bump moby vendor and dependencies
Upstream-commit: 02e8dc6
Component: cli
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@thaJeztah@cpuguy83@codecov-io@vdemeester@dnephin@GordonTheTurtle