[sssd PR#561][opened] DYNDNS: Retry also on timeouts

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/561
Author: jhrozek
 Title: #561: DYNDNS: Retry also on timeouts
Action: opened

PR body:
"""
There is the dyndns_server option that is supposed to make it possible for
the admin to select a server to update DNS with if the server detected by
nsupdate does not work. The fallback works OK for the case where nsupdate
fails with a non-zero return code, but doesn't work for the case where
nsupdate times out.

This patch extends the retry condition to also fallback to the
dyndns_server directive if nsupdate return ERR_DYNDNS_TIMEOUT.
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/561/head:pr561
git checkout pr561

[sssd PR#561][comment] DYNDNS: Retry also on timeouts

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/561
Title: #561: DYNDNS: Retry also on timeouts

jhrozek commented:
"""
retest this please
"""

See the full comment at https://github.com/SSSD/sssd/pull/561#issuecomment-385762586

[sssd PR#561][comment] DYNDNS: Retry also on timeouts

[fidencio]

URL: https://github.com/SSSD/sssd/pull/561
Title: #561: DYNDNS: Retry also on timeouts

fidencio commented:
"""
Patches look good. I'm running the internal CI just for the sake of the process and I'll add the accepted label as soon as I get the results.
"""

See the full comment at https://github.com/SSSD/sssd/pull/561#issuecomment-386209311

[sssd PR#561][comment] DYNDNS: Retry also on timeouts

[fidencio]

URL: https://github.com/SSSD/sssd/pull/561
Title: #561: DYNDNS: Retry also on timeouts

fidencio commented:
"""
Ack!
CI: http://vm-031.abc.idm.lab.eng.brq.redhat.com/logs/job/88/42/summary.html
"""

See the full comment at https://github.com/SSSD/sssd/pull/561#issuecomment-386266504

[sssd PR#561][+Accepted] DYNDNS: Retry also on timeouts

[fidencio]

URL: https://github.com/SSSD/sssd/pull/561
Title: #561: DYNDNS: Retry also on timeouts

Label: +Accepted

[sssd PR#561][comment] DYNDNS: Retry also on timeouts

[fidencio]

URL: https://github.com/SSSD/sssd/pull/561
Title: #561: DYNDNS: Retry also on timeouts

fidencio commented:
"""
Ack!
CI: http://vm-031.$%7Babc%7D/logs/job/88/42/summary.html
"""

See the full comment at https://github.com/SSSD/sssd/pull/561#issuecomment-386266504

[sssd PR#561][comment] DYNDNS: Retry also on timeouts

[fidencio]

URL: https://github.com/SSSD/sssd/pull/561
Title: #561: DYNDNS: Retry also on timeouts

fidencio commented:
"""
master:
 65034a7
 b57dfac
"""

See the full comment at https://github.com/SSSD/sssd/pull/561#issuecomment-386419595

[sssd PR#561][closed] DYNDNS: Retry also on timeouts

[fidencio]

URL: https://github.com/SSSD/sssd/pull/561
Author: jhrozek
 Title: #561: DYNDNS: Retry also on timeouts
Action: closed

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/561/head:pr561
git checkout pr561

[sssd PR#561][+Pushed] DYNDNS: Retry also on timeouts

[fidencio]

URL: https://github.com/SSSD/sssd/pull/561
Title: #561: DYNDNS: Retry also on timeouts

Label: +Pushed

[sssd PR#561][-Accepted] DYNDNS: Retry also on timeouts

[fidencio]

URL: https://github.com/SSSD/sssd/pull/561
Title: #561: DYNDNS: Retry also on timeouts

Label: -Accepted