Skip to content

Add a note about moby/moby#46603 in v25.0 release notes#19173

Merged
dvdksn merged 1 commit intodocker:mainfrom
akerouanton:release-notes-moby-46603
Jan 22, 2024
Merged

Add a note about moby/moby#46603 in v25.0 release notes#19173
dvdksn merged 1 commit intodocker:mainfrom
akerouanton:release-notes-moby-46603

Conversation

@akerouanton
Copy link
Member

Signed-off-by: Albin Kerouanton <albinker@gmail.com>
@akerouanton akerouanton requested a review from dvdksn as a code owner January 22, 2024 08:39
@netlify
Copy link

netlify bot commented Jan 22, 2024

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit c1c034e
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/65ae29bc24595800087497f6
😎 Deploy Preview https://deploy-preview-19173--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 site configuration.

@thaJeztah thaJeztah added this to the Engine v25.0.0 milestone Jan 22, 2024
@thaJeztah thaJeztah added area/engine Issue affects Docker engine/daemon area/networking Relates to anything around networking labels Jan 22, 2024
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.

LGTM

@dvdksn dvdksn merged commit c6dcf2f into docker:main Jan 22, 2024
@akerouanton akerouanton deleted the release-notes-moby-46603 branch January 22, 2024 09:30
@thaJeztah
Copy link
Member

@akerouanton @dvdksn do we have a section describing internal? If so, perhaps we should add a description what it does (including this)?

@akerouanton
Copy link
Member Author

If so, perhaps we should add a description what it does (including this)?

The way the default routing table is set up is an implementation detail and could be modified in the future, so I'm a bit on the fence. At least, we should mention that if we document it.

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