-
Notifications
You must be signed in to change notification settings - Fork 7.7k
New engine options, "allow-direct-routing" and "trusted_host_interfaces" #22601
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
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
70f9fba
to
824c257
Compare
Docker host. This can be achieved using static routes, Border Gateway Protocol | ||
(BGP), or any other means appropriate for your network. For example, within |
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.
- opened a quick PR to make vale happy on this acronym; vale: add "BGP" (Border Gateway Protocol) to acronyms #22731
Linux bridge network, use daemon option `--allow-direct-routing` or the | ||
equivalent `"allow-direct-routing": true` in `/etc/docker/daemon.json`. |
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.
Nit: not sure if we're consistent on this, but generally, I think we recommend users to use daemon.json
for configs, so perhaps we should document that as the first option to set this (with the --allow-direct-routing
command-line option as alternative)
the host's loopback interface: | ||
|
||
```console | ||
docker run -d --ip 192.0.2.100 -p 127.0.0.1:8080:80 nginx |
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.
Can you add a prompt here?
docker run -d --ip 192.0.2.100 -p 127.0.0.1:8080:80 nginx | |
$ docker run -d --ip 192.0.2.100 -p 127.0.0.1:8080:80 nginx |
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
left two comments / suggestions, but looks good after those are looked at, thanks!
Signed-off-by: Rob Murray <rob.murray@docker.com>
824c257
to
b11da57
Compare
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
I think this can be merged now, 28.2 shipped last week (https://github.com/moby/moby/releases/tag/v28.2.0). |
Description
This update is for moby 28.2.0 - do not merge until it ships.- 28.2 has shipped now.Add description for daemon option
--allow-direct-routing
and network optioncom.docker.network.bridge.trusted_host_interfaces
.Related issues or tickets
moby/moby#49832
Reviews