8000 New engine options, "allow-direct-routing" and "trusted_host_interfaces" by robmry · Pull Request #22601 · docker/docs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged
merged 1 commit into from
Jun 3, 2025

Conversation

robmry
Copy link
Contributor
@robmry robmry commented May 7, 2025

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 option com.docker.network.bridge.trusted_host_interfaces.

Related issues or tickets

moby/moby#49832

Reviews

  • Technical review
  • Editorial review
  • Product review

Copy link
netlify bot commented May 7, 2025

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit b11da57
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6839b05b298a0b0008a2415d
😎 Deploy Preview https://deploy-preview-22601--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added area/engine Issue affects Docker engine/daemon area/networking Relates to anything around networking labels May 7, 2025
@robmry robmry force-pushed the moby_trusted_interfaces branch from 70f9fba to 824c257 Compare May 30, 2025 08:32
ArthurFlag
ArthurFlag previously approved these changes May 30, 2025
Comment on lines 154 to 155
Docker host. This can be achieved using static routes, Border Gateway Protocol
(BGP), or any other means appropriate for your network. For example, within
Copy link
Member

Choose a reason for hiding this comment

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

Comment on lines 166 to 167
Linux bridge network, use daemon option `--allow-direct-routing` or the
equivalent `"allow-direct-routing": true` in `/etc/docker/daemon.json`.
Copy link
Member

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
Copy link
Member

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?

Suggested change
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

thaJeztah
thaJeztah previously approved these changes May 30, 2025
Copy link
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.

LGTM

left two comments / suggestions, but looks good after those are looked at, thanks!

Signed-off-by: Rob Murray <rob.murray@docker.com>
@robmry robmry dismissed stale reviews from thaJeztah and ArthurFlag via b11da57 May 30, 2025 13:19
@robmry robmry force-pushed the moby_trusted_interfaces branch from 824c257 to b11da57 Compare May 30, 2025 13:19
Copy link
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.

LGTM

@robmry
Copy link
Contributor Author
robmry commented Jun 2, 2025

I think this can be merged now, 28.2 shipped last week (https://github.com/moby/moby/releases/tag/v28.2.0).

@ArthurFlag ArthurFlag merged commit 62d2509 into docker:main Jun 3, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/engine Issue affects Docker engine/daemon area/networking Relates to anything around networking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0