Skip to content

Add --host-gateway-ip to the dockerd manpage#48988

Merged
thaJeztah merged 2 commits intomoby:masterfrom
robmry:dockerd_manpage_host-gateway-ip
Nov 29, 2024
Merged

Add --host-gateway-ip to the dockerd manpage#48988
thaJeztah merged 2 commits intomoby:masterfrom
robmry:dockerd_manpage_host-gateway-ip

Conversation

@robmry
Copy link
Copy Markdown
Contributor

@robmry robmry commented Nov 29, 2024

- What I did

Add --host-gateway-ip to moby's copy of the dockerd manpage, and put --help before --host.

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@robmry robmry added kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. area/networking Networking labels Nov 29, 2024
@robmry robmry added this to the 28.0.0 milestone Nov 29, 2024
@robmry robmry self-assigned this Nov 29, 2024
@robmry robmry requested a review from thaJeztah November 29, 2024 17:20
@thaJeztah
Copy link
Copy Markdown
Member

Oh! Interesting, why did jenkins start failing on this? I think some updates were rolled out; wondering if related 🤔

Stacktrace

=== RUN   TestCreateMultipleNetworks
time="2024-11-29T17:26:55Z" level=error msg="Running modprobe br_netfilter failed with message: " error="exec: \"modprobe\": executable file not found in $PATH"
    bridge_linux_test.go:484: Failed to create bridge: loadBridgeNetFilterModule failed: cannot restrict inter-container communication: modprobe br_netfilter failed: exec: "modprobe": executable file not found in $PATH

@robmry
Copy link
Copy Markdown
Contributor Author

robmry commented Nov 29, 2024

Oh! Interesting, why did jenkins start failing on this? I think some updates were rolled out; wondering if related 🤔

Hmm, that is odd ... more so because it doesn't look like the test disables ICC or docker-proxy, so it shouldn't want to load the module. We just changed that code but, although it might try to modprobe in more circumstances, it still shouldn't try unless necessary.

Edit: the test does disable ICC (by not explicitly enabling it). So at least that makes sense.

@thaJeztah
Copy link
Copy Markdown
Member

@thaJeztah
Copy link
Copy Markdown
Member

thaJeztah commented Nov 29, 2024

OHMAN went a bit down the rabbit-hole on that one;

Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

either way; this pr LGTM 👍

@thaJeztah thaJeztah merged commit 321f9c2 into moby:master Nov 29, 2024
@robmry robmry deleted the dockerd_manpage_host-gateway-ip branch December 3, 2024 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/networking Networking kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. status/3-docs-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants