Skip to content

Conversation

@robmry
Copy link
Contributor

@robmry robmry commented Sep 12, 2025

Description

These changes aren't moby-29 specific - but 29.0 will be ready soon, and the networking docs moved around a bit (so, less chance of conflict confusion this way).

@robmry robmry requested a review from ArthurFlag as a code owner September 12, 2025 18:48
@github-actions github-actions bot added area/engine Issue affects Docker engine/daemon area/networking Relates to anything around networking labels Sep 12, 2025
@github-actions github-actions bot added area/compose Relates to docker-compose.yml spec or docker-compose binary area/install Relates to installing a product area/hub Issue affects Docker Hub area/build Relates to Dockerfiles or docker build command area/desktop Issue affects a desktop edition of Docker. E.g docker for mac area/cli Relates to the CLI client area/security area/get-started Relates to get started and onboarding docs area/subscription Relates to Docker pricing and subscription area/extensions Relates to Docker Extensions area/api Relates to Docker API docs area/contrib Relates to the Docker style guide and contribution guidelines area/release Relates to CI or deployment area/admin Relates to Docker Admin area/scout Relates to Docker Scout hugo Updates related to hugo dependencies Vendoring, packaging updates area/billing area/config area/tests labels Nov 13, 2025
@robmry robmry marked this pull request as draft November 13, 2025 09:45
@robmry robmry changed the base branch from moby29 to main November 13, 2025 09:45
@robmry robmry marked this pull request as ready for review November 13, 2025 09:45
@robmry robmry requested a review from dvdksn as a code owner November 13, 2025 09:45
@robmry
Copy link
Contributor Author

robmry commented Nov 13, 2025

@dvdksn - we shipped 29.0.0, so this can be reviewed/merged now.

(The bonkers list of reviewers and labels came about because I rebased on main, while the PR was still pointing at the moby29 branch.)

@dvdksn
Copy link
Contributor

dvdksn commented Nov 13, 2025

Looks like the build has hung...

@netlify
Copy link

netlify bot commented Nov 13, 2025

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 525496d
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6915afbd22b92e0008203147
😎 Deploy Preview https://deploy-preview-23409--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.

Copy link
Contributor

@dvdksn dvdksn left a comment

Choose a reason for hiding this comment

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

Looks great, just a minor suggestion for clarity

Comment on lines 294 to 297
By default, if NAT is enabled for a bridge network, outgoing packets from
containers are masqueraded. This means the source address of packets
leaving the Docker host is changed to an address on the host interface
the packet is sent on.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
By default, if NAT is enabled for a bridge network, outgoing packets from
containers are masqueraded. This means the source address of packets
leaving the Docker host is changed to an address on the host interface
the packet is sent on.
NAT is enabled by default for bridge networks, meaning outgoing packets
from containers are masqueraded. The source address of packets leaving
the Docker host is changed to an address on the host interface the packet
is sent on.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you - done.

@dvdksn dvdksn merged commit b8288c5 into docker:main Nov 13, 2025
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/admin Relates to Docker Admin area/ai area/api Relates to Docker API docs area/billing area/build Relates to Dockerfiles or docker build command area/cli Relates to the CLI client area/compose Relates to docker-compose.yml spec or docker-compose binary area/config area/contrib Relates to the Docker style guide and contribution guidelines area/copilot area/desktop Issue affects a desktop edition of Docker. E.g docker for mac area/dhi area/engine Issue affects Docker engine/daemon area/extensions Relates to Docker Extensions area/get-started Relates to get started and onboarding docs area/guides area/hub Issue affects Docker Hub area/install Relates to installing a product area/networking Relates to anything around networking area/offload area/release Relates to CI or deployment area/scout Relates to Docker Scout area/security area/subscription Relates to Docker pricing and subscription area/tests ci ci updates dependencies Vendoring, packaging updates hugo Updates related to hugo

Projects

None yet

2 participants