SSSD man: man_dns_resolver_parameter_modification #5645
Conversation
DDDas7
force-pushed
the
man_dns_resolver_parameter_modification
branch
from
45600ee
to
de7d32c
Compare
DDDas7
force-pushed
the
man_dns_resolver_parameter_modification
branch
from
de7d32c
to
25843fc
Compare
Adding parameter dns_resolver_server_timeout and dns_resolver_op_timeout in sssd.conf Resolves: SSSD#5616
DDDas7
force-pushed
the
man_dns_resolver_parameter_modification
branch
from
25843fc
to
9ab6cc0
Compare
src/man/sssd.conf.5.xml
Outdated
| <para> | ||
| Defines the amount of time (in milliseconds) | ||
| SSSD would try to talk to DNS server before | ||
| trying next DNS server. |
There was a problem hiding this comment.
Hi, I read this a couple times but it's not really clear to me what it means.
I interpret this as SSSD would try to talk to a DNS server for 1 second before trying a next DNS server, but why would SSSD only communicate with a DNS server for 1 second? Does it mean SSSD connects to a DNS server but only waits for a response for 1 second?
There was a problem hiding this comment.
I think that the timer is only used to resolve the DNS server, once it's resolved the timer no longer applies.
There was a problem hiding this comment.
I have rephrased the complete sentence. Can you pls check the latest changes.
Also I have split the dns_resolver_* parameters from “failover.xml” and created a new file “dns_options.xml” to sync the parameter definition since currently the definition for the parameter “dns_resolver_timeout” is different in files “failover.xml” and “sssd.conf.5.xml”.
It will help maintaining the dns_resolver_* parameters definition from a single file.
DDDas7
changed the title
Adding parameter dns_resolver_server_timeout and dns_resolver_op_timeout in sssd.conf Resolves: SSSD#5616
DDDas7
changed the title
DDDas7
force-pushed
the
man_dns_resolver_parameter_modification
branch
from
6a60c15
to
addf2a6
Compare
Adding parameter dns_resolver_server_timeout and dns_resolver_op_timeout in sssd.conf Resolves: SSSD#5616
DDDas7
force-pushed
the
man_dns_resolver_parameter_modification
branch
from
addf2a6
to
5c422e6
Compare
|
LGTM! @justin-stephenson can you do a final review? |
Ack, thanks Deepak ! |
Adding parameter dns_resolver_server_timeout
and dns_resolver_op_timeout in sssd.conf
Resolves: #5616