Skip to content

Conversation

@vvoland
Copy link
Collaborator

@vvoland vvoland commented Dec 18, 2025

@vvoland vvoland self-assigned this Dec 18, 2025
@vvoland vvoland requested review from a team and thaJeztah as code owners December 18, 2025 11:55
Comment on lines 1284 to 1285
`host-gateway`. Docker Desktop automatically resolves this hostname, see
[Explore networking features](https://docs.docker.com/desktop/features/networking/networking-how-tos/#i-want-to-connect-from-a-container-to-a-service-on-the-host).
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I couldn't find this section in docs anymore.

@robmry @dvdksn do we have some other page we could use here?

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

But this CLI docs page has a better description https://docs.docker.com/reference/cli/dockerd/#configure-host-gateway-ip

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ohh right! I updated and put them both. WDYT?

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
@vvoland vvoland changed the title docs/container: Remove dead docs reference docs/container: Fix dead docs reference Dec 18, 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

@thaJeztah thaJeztah added this to the 29.2.0 milestone Dec 18, 2025
@thaJeztah thaJeztah merged commit f135652 into docker:master Dec 18, 2025
156 of 158 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants