Skip to content

[20.10] vendor: github.com/moby/libnetwork dcdf8f176d1e13ad719e913e796fb698d846de98#44476

Merged
thaJeztah merged 1 commit into
moby:20.10from
sbuckfelder:20.10_UPDATE
Dec 8, 2022
Merged

[20.10] vendor: github.com/moby/libnetwork dcdf8f176d1e13ad719e913e796fb698d846de98#44476
thaJeztah merged 1 commit into
moby:20.10from
sbuckfelder:20.10_UPDATE

Conversation

@sbuckfelder

@sbuckfelder sbuckfelder commented Nov 16, 2022

Copy link
Copy Markdown

Signed-off-by: Scott Buckfelder [email protected]

Issue : #44423
fixes #43054

Merges in commit moby/libnetwork#2668 which is a backport of functionality.

Note For Reviewers : I also updated the hack/dockerfile/install/proxy.installer file to now reference moby/libnetwork as opposed to docker/libnetwork. I'm not terribly confident this was the correct thing to do.

@sbuckfelder
sbuckfelder requested a review from tianon as a code owner November 16, 2022 14:56

@neersighted neersighted left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Updating repository references is fine, but the import path is still docker/libnetwork and cannot change.

Everything but the git clone needs to revert to the old name.

@neersighted

Copy link
Copy Markdown
Member

Also, could you please rephrase your commit message to conform to the repo convention for vendoring? e.g. 2570784

@sbuckfelder sbuckfelder changed the title Revert: Added API to set ephemeral port allocator range [20.10] vendor: github.com/moby/libnetwork dcdf8f176d1e13ad719e913e796fb698d846de98 Nov 17, 2022
@sbuckfelder

Copy link
Copy Markdown
Author

Also, could you please rephrase your commit message to conform to the repo convention for vendoring? e.g. 2570784

Updated in new rev

@sbuckfelder

Copy link
Copy Markdown
Author

Updating repository references is fine, but the import path is still docker/libnetwork and cannot change.

Everything but the git clone needs to revert to the old name.

I believe I was able to follow this advice in the new rev, please let me know if not.

@neersighted

Copy link
Copy Markdown
Member

This looks good from a changes made standpoint, but fails the vendoring checks. Did you vendor by hand instead of using vndr?

The result of vndr differs

You should run hack/vendor.sh every time you touch vendoring; fix that and this should be good to go.

@thaJeztah thaJeztah added status/2-code-review area/networking Networking status/failing-ci Indicates that the PR in its current state fails the test suite kind/bugfix PR's that fix bugs labels Nov 17, 2022
@thaJeztah thaJeztah added this to the 20.10.22 milestone Nov 17, 2022
…6fb698d846de98

Co-authored-by: Sebastiaan van Stijn <[email protected]>
Signed-off-by: Scott Buckfelder <[email protected]>
@thaJeztah

Copy link
Copy Markdown
Member

I amended the commit and ran vendor

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM (assuming CI is happy)

@thaJeztah thaJeztah removed the status/failing-ci Indicates that the PR in its current state fails the test suite label Nov 23, 2022
@thaJeztah
thaJeztah merged commit 6112b23 into moby:20.10 Dec 8, 2022
paralin added a commit to skiffos/buildroot that referenced this pull request Jan 13, 2023
Bug fixes and enhancements

 - Improve error message when attempting to pull an unsupported image format or OCI artifact (moby/moby#44413, moby/moby#44569)
 - Fix an issue where the host's ephemeral port-range was ignored when selecting random ports for containers (moby/moby#44476).
 - Fix ssh: parse error in message type 27 errors during docker build on hosts using OpenSSH 8.9 or above (moby/moby#3862).
 - seccomp: block socket calls to AF_VSOCK in default profile (moby/moby#44564).

https://github.com/moby/moby/releases/tag/v20.10.22

Signed-off-by: Christian Stewart <[email protected]>
paralin added a commit to skiffos/buildroot that referenced this pull request Jan 13, 2023
Bug fixes and enhancements

 - Improve error message when attempting to pull an unsupported image format or OCI artifact (moby/moby#44413, moby/moby#44569)
 - Fix an issue where the host's ephemeral port-range was ignored when selecting random ports for containers (moby/moby#44476).
 - Fix ssh: parse error in message type 27 errors during docker build on hosts using OpenSSH 8.9 or above (moby/moby#3862).
 - seccomp: block socket calls to AF_VSOCK in default profile (moby/moby#44564).

https://github.com/moby/moby/releases/tag/v20.10.22

Signed-off-by: Christian Stewart <[email protected]>
arnout pushed a commit to buildroot/buildroot that referenced this pull request Jan 14, 2023
Bug fixes and enhancements

 - Improve error message when attempting to pull an unsupported image format or OCI artifact (moby/moby#44413, moby/moby#44569)
 - Fix an issue where the host's ephemeral port-range was ignored when selecting random ports for containers (moby/moby#44476).
 - Fix ssh: parse error in message type 27 errors during docker build on hosts using OpenSSH 8.9 or above (moby/moby#3862).
 - seccomp: block socket calls to AF_VSOCK in default profile (moby/moby#44564).

https://github.com/moby/moby/releases/tag/v20.10.22

Signed-off-by: Christian Stewart <[email protected]>
Signed-off-by: Peter Korsgaard <[email protected]>
arnout pushed a commit to buildroot/buildroot that referenced this pull request Jan 15, 2023
Bug fixes and enhancements

 - Improve error message when attempting to pull an unsupported image format or OCI artifact (moby/moby#44413, moby/moby#44569)
 - Fix an issue where the host's ephemeral port-range was ignored when selecting random ports for containers (moby/moby#44476).
 - Fix ssh: parse error in message type 27 errors during docker build on hosts using OpenSSH 8.9 or above (moby/moby#3862).
 - seccomp: block socket calls to AF_VSOCK in default profile (moby/moby#44564).

https://github.com/moby/moby/releases/tag/v20.10.22

Signed-off-by: Christian Stewart <[email protected]>
Signed-off-by: Peter Korsgaard <[email protected]>
(cherry picked from commit de51efc)
Signed-off-by: Peter Korsgaard <[email protected]>
arnout pushed a commit to buildroot/buildroot that referenced this pull request Jan 15, 2023
Bug fixes and enhancements

 - Improve error message when attempting to pull an unsupported image format or OCI artifact (moby/moby#44413, moby/moby#44569)
 - Fix an issue where the host's ephemeral port-range was ignored when selecting random ports for containers (moby/moby#44476).
 - Fix ssh: parse error in message type 27 errors during docker build on hosts using OpenSSH 8.9 or above (moby/moby#3862).
 - seccomp: block socket calls to AF_VSOCK in default profile (moby/moby#44564).

https://github.com/moby/moby/releases/tag/v20.10.22

Signed-off-by: Christian Stewart <[email protected]>
Signed-off-by: Peter Korsgaard <[email protected]>
(cherry picked from commit de51efc)
Signed-off-by: Peter Korsgaard <[email protected]>
ccrisan pushed a commit to ccrisan/thingos that referenced this pull request Jun 26, 2025
Bug fixes and enhancements

 - Improve error message when attempting to pull an unsupported image format or OCI artifact (moby/moby#44413, moby/moby#44569)
 - Fix an issue where the host's ephemeral port-range was ignored when selecting random ports for containers (moby/moby#44476).
 - Fix ssh: parse error in message type 27 errors during docker build on hosts using OpenSSH 8.9 or above (moby/moby#3862).
 - seccomp: block socket calls to AF_VSOCK in default profile (moby/moby#44564).

https://github.com/moby/moby/releases/tag/v20.10.22

Signed-off-by: Christian Stewart <[email protected]>
Signed-off-by: Peter Korsgaard <[email protected]>
ccrisan pushed a commit to qtoggle/qtoggleos that referenced this pull request Oct 21, 2025
Bug fixes and enhancements

 - Improve error message when attempting to pull an unsupported image format or OCI artifact (moby/moby#44413, moby/moby#44569)
 - Fix an issue where the host's ephemeral port-range was ignored when selecting random ports for containers (moby/moby#44476).
 - Fix ssh: parse error in message type 27 errors during docker build on hosts using OpenSSH 8.9 or above (moby/moby#3862).
 - seccomp: block socket calls to AF_VSOCK in default profile (moby/moby#44564).

https://github.com/moby/moby/releases/tag/v20.10.22

Signed-off-by: Christian Stewart <[email protected]>
Signed-off-by: Peter Korsgaard <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants