Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add docs for --dns-search=.#8839

Merged
crosbymichael merged 2 commits into
moby:masterfrom
irachex:dns-search-doc
Oct 31, 2014
Merged

Add docs for --dns-search=.#8839
crosbymichael merged 2 commits into
moby:masterfrom
irachex:dns-search-doc

Conversation

@irachex

@irachex irachex commented Oct 29, 2014

Copy link
Copy Markdown
Contributor

PR #6720 introduce that use --dns-search=. will not set search in /etc/resolv.conf.

Comment thread docs/sources/articles/networking.md Outdated

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.

search

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.

Use --dns-search=. if you don't wish to set the search domain.

@jamtur01

Copy link
Copy Markdown
Contributor

Thanks for this. Can you please update based on feedback.

@irachex

irachex commented Oct 30, 2014

Copy link
Copy Markdown
Contributor Author

@jamtur01 Thanks for your review. updated

PR 6720 introduce that use `--dns-search=.` will not set `search` in `/etc/resolv.conf`.

Signed-off-by: Huayi Zhang <[email protected]>
@jamtur01

Copy link
Copy Markdown
Contributor

LGTM Ping @SvenDowideit @fredlf

@fredlf

fredlf commented Oct 30, 2014

Copy link
Copy Markdown
Contributor

LGTM and many thanks for the contribution.

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.

This needs a matching change in the code.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed eaa050f

Signed-off-by: Huayi Zhang <[email protected]>
@SvenDowideit

Copy link
Copy Markdown
Contributor

Docs LGTM - @crosbymichael @vieux @tiborvass @jfrazelle

@jessfraz

Copy link
Copy Markdown
Contributor

LGTM

1 similar comment
@crosbymichael

Copy link
Copy Markdown
Contributor

LGTM

crosbymichael added a commit that referenced this pull request Oct 31, 2014
@crosbymichael crosbymichael merged commit 0f9f5f3 into moby:master Oct 31, 2014
SvenDowideit pushed a commit to SvenDowideit/docker that referenced this pull request Nov 6, 2014
Add docs for --dns-search=.
(cherry picked from commit 0f9f5f3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants