Skip to content

docs: document special host-gateway value for add-host#4399

Merged
thaJeztah merged 1 commit into
docker:masterfrom
dvdksn:docs/host-gateway
Aug 8, 2023
Merged

docs: document special host-gateway value for add-host#4399
thaJeztah merged 1 commit into
docker:masterfrom
dvdksn:docs/host-gateway

Conversation

@dvdksn

@dvdksn dvdksn commented Jul 3, 2023

Copy link
Copy Markdown
Contributor

Signed-off-by: David Karlsson [email protected]

- What I did

Add some docs on --add-host host.docker.internal:host-gateway and dockerd --host-gateway-ip

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

PXL_20230617_072054134

@codecov-commenter

codecov-commenter commented Jul 3, 2023

Copy link
Copy Markdown

Codecov Report

Merging #4399 (299925f) into master (dc2eb3b) will increase coverage by 0.01%.
Report is 56 commits behind head on master.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4399      +/-   ##
==========================================
+ Coverage   59.38%   59.39%   +0.01%     
==========================================
  Files         288      288              
  Lines       24749    24782      +33     
==========================================
+ Hits        14696    14720      +24     
- Misses       9169     9175       +6     
- Partials      884      887       +3     

@dvdksn

dvdksn commented Aug 8, 2023

Copy link
Copy Markdown
Contributor Author

@thaJeztah does this LGTY?

Comment thread docs/reference/commandline/run.md Outdated
Comment thread docs/reference/commandline/run.md
@dvdksn
dvdksn force-pushed the docs/host-gateway branch from 0d92bcd to 299925f Compare August 8, 2023 15:37

@thaJeztah thaJeztah 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, thanks!

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.

3 participants