URL:
https://github.com/SSSD/sssd/pull/768
Title: #768: providers/ipa: Changed default service search base
pbrezina commented:
"""
The manpage change is not correct because it says the default is cn=ipservices for all
provider, but it is the default only for the IPA provider.
Other search bases have different default with ipa providers as well, but it is not
documented (e.g. `ldap_netgroup_search_base`). For this reason, I do not think any man
page change is necessary. But if you insist on documenting it, then see for example
`ldap_host_search_base`:
```
<varlistentry>
<term>ldap_host_search_base (string)</term>
<listitem>
<para>
Optional. Use the given string as search base for
host objects.
</para>
<para>
See <quote>ldap_search_base</quote> for
information about configuring multiple search
bases.
</para>
<para>
Default: the value of
<emphasis>ldap_search_base</emphasis>
</para>
</listitem>
</varlistentry>
```
Please, either drop the second patch or simplify it like this (mentioning that for ipa
provider the default is different).
"""
See the full comment at
https://github.com/SSSD/sssd/pull/768#issuecomment-487526788