Skip to content

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

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

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

Conversation

@dvdksn
Copy link
Copy Markdown
Contributor

@dvdksn dvdksn commented Jul 3, 2023

Signed-off-by: David Karlsson 35727626+dvdksn@users.noreply.github.com

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

codecov-commenter commented Jul 3, 2023

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

dvdksn commented Aug 8, 2023

@thaJeztah does this LGTY?

Comment thread docs/reference/commandline/run.md Outdated
Comment thread docs/reference/commandline/run.md
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
@dvdksn dvdksn force-pushed the docs/host-gateway branch from 0d92bcd to 299925f Compare August 8, 2023 15:37
Copy link
Copy Markdown
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, 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