-
Notifications
You must be signed in to change notification settings - Fork 18.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add docs for --dns-search=. #8839
Conversation
@@ -170,6 +170,7 @@ Four different options affect container domain name services. | |||
When a container process attempts to access `host` and the search | |||
domain `example.com` is set, for instance, the DNS logic will not | |||
only look up `host` but also `host.example.com`. | |||
Use `--dns-serach=.` to not set search domain. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
search
There was a problem hiding this comment.
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.
Thanks for this. Can you please update based on feedback. |
c890c2b
to
4aacb25
Compare
@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 <irachex@gmail.com>
4aacb25
to
36ffbd7
Compare
LGTM Ping @SvenDowideit @fredlf |
LGTM and many thanks for the contribution. |
@@ -64,7 +64,7 @@ docker-create - Create a new container | |||
Add a host device to the container (e.g. --device=/dev/sdc:/dev/xvdc:rwm) | |||
|
|||
**--dns-search**=[] | |||
Set custom DNS search domains | |||
Set custom DNS search domains (Use --dns-search=. if you don't wish to set the search domain) |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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 <irachex@gmail.com>
Docs LGTM - @crosbymichael @vieux @tiborvass @jfrazelle |
LGTM |
1 similar comment
LGTM |
Add docs for --dns-search=.
Add docs for --dns-search=. (cherry picked from commit 0f9f5f3)
PR #6720 introduce that use
--dns-search=.
will not setsearch
in/etc/resolv.conf
.