Skip to content

docs/container: Fix dead docs reference - #6715

Merged
thaJeztah merged 1 commit into
docker:masterfrom
vvoland:work-docs
Dec 18, 2025
Merged

docs/container: Fix dead docs reference#6715
thaJeztah merged 1 commit into
docker:masterfrom
vvoland:work-docs

Conversation

@vvoland

Copy link
Copy Markdown
Collaborator

@vvolandvvoland self-assigned this Dec 18, 2025
@vvoland
vvoland requested review from a team and thaJeztah as code ownersDecember 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
Copy Markdown
CollaboratorAuthor

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
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
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
Copy Markdown
CollaboratorAuthor

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
Copy Markdown

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>
@vvolandvvoland changed the title docs/container: Remove dead docs referencedocs/container: Fix dead docs referenceDec 18, 2025

@thaJeztahthaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@thaJeztahthaJeztah added this to the 29.2.0 milestone Dec 18, 2025
@thaJeztah
thaJeztah merged commit f135652 into docker:masterDec 18, 2025
156 of 158 checks passed
Sign up for freeto 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

@vvoland@codecov-commenter@thaJeztah@robmry