[sssd PR#721][opened] AD/IPA: Reset subdomain service name, not domain name

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/721
Author: jhrozek
 Title: #721: AD/IPA: Reset subdomain service name, not domain name
Action: opened

PR body:
"""
Related: https://pagure.io/SSSD/sssd/issue/3911

Since commit 778f241e78241b0d6b8734148175f8dee804f494 the subdomain fail 
over services use the "sd_" prefix. This was done to make it easier, until
the whole failover design works better with subdomains, to see which
services belong to the main domain from tools.

However, some parts of the code would still just use the domain name for 
the failover service, which meant the service was not found, notably when
trying to reset services:

(Thu Dec 13 05:29:31 2018) [sssd[be[testrelm.test]]]
[ipa_srv_ad_acct_retried] (0x0400): Subdomain re-set, will retry lookup
(Thu Dec 13 05:29:31 2018) [sssd[be[testrelm.test]]] [be_fo_reset_svc]
(0x1000): Resetting all servers in service ipaad2016.test
(Thu Dec 13 05:29:31 2018) [sssd[be[testrelm.test]]] [be_fo_reset_svc]
(0x0080): Cannot retrieve service [ipaad2016.test]

This patch switches to reading the service names from the ad_options and 
the sdap_service structures that are contained within ad_options.
"""

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

[sssd PR#721][+Accepted] AD/IPA: Reset subdomain service name, not domain name

[thalman]

URL: https://github.com/SSSD/sssd/pull/721
Title: #721: AD/IPA: Reset subdomain service name, not domain name

Label: +Accepted

[sssd PR#721][comment] AD/IPA: Reset subdomain service name, not domain name

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/721
Title: #721: AD/IPA: Reset subdomain service name, not domain name

jhrozek commented:
"""
* master:
 * aaaa9a3e836e7b7af1ff0fc5058ddddfeef120a8
 * b3285f9f8a5eac3e4e70ed3bd6b74c15ad806e9e
"""

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

[sssd PR#721][closed] AD/IPA: Reset subdomain service name, not domain name

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/721
Author: jhrozek
 Title: #721: AD/IPA: Reset subdomain service name, not domain name
Action: closed

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

[sssd PR#721][+Pushed] AD/IPA: Reset subdomain service name, not domain name

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/721
Title: #721: AD/IPA: Reset subdomain service name, not domain name

Label: +Pushed