[sssd PR#479][opened] MAN: Document that auth and access IPA and AD providers rely on id_provider being set to the same type

[jhrozek]

   URL: https://github.com/SSSD/sssd/pull/479
Author: jhrozek
 Title: #479: MAN: Document that auth and access IPA and AD providers rely on id_provider
being set to the same type
Action: opened

PR body:
"""
Resolves: https://pagure.io/SSSD/sssd/issue/3547

The IPA and AD auth and access providers rely (often for performance
reasons) on certain properties that are set during the user or group
resolution with the same provider type.

However, there are users who wish to combine different provider types,
typically to use identities from local UNIX files but authenticate against
a remote server. We should discourage that in our documentation

But at the same time, I think flat our failing would be too harsh...I was
also thinking about printing a DEBUG message during initialization, if the
reviewer thinks this is a good idea, I'll do that as well.
"""

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

[sssd PR#479][comment] MAN: Document that auth and access IPA and AD providers rely on id_provider being set to the same type

[fidencio]

  URL: https://github.com/SSSD/sssd/pull/479
Title: #479: MAN: Document that auth and access IPA and AD providers rely on id_provider
being set to the same type

fidencio commented:
"""
This patch is good as it is, no need of a debug message IMO.
"""

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

[sssd PR#479][+Accepted] MAN: Document that auth and access IPA and AD providers rely on id_provider being set to the same type

[fidencio]

  URL: https://github.com/SSSD/sssd/pull/479
Title: #479: MAN: Document that auth and access IPA and AD providers rely on id_provider
being set to the same type

Label: +Accepted

[sssd PR#479][comment] MAN: Document that auth and access IPA and AD providers rely on id_provider being set to the same type

[justin-stephenson]

  URL: https://github.com/SSSD/sssd/pull/479
Title: #479: MAN: Document that auth and access IPA and AD providers rely on id_provider
being set to the same type

justin-stephenson commented:
"""
Added a wording suggestion(feel free to take it or leave it) but otherwise looks good to
me.
"""

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

[sssd PR#479][-Accepted] MAN: Document that auth and access IPA and AD providers rely on id_provider being set to the same type

[lslebodn]

  URL: https://github.com/SSSD/sssd/pull/479
Title: #479: MAN: Document that auth and access IPA and AD providers rely on id_provider
being set to the same type

Label: -Accepted

[sssd PR#479][+Changes requested] MAN: Document that auth and access IPA and AD providers rely on id_provider being set to the same type

[lslebodn]

  URL: https://github.com/SSSD/sssd/pull/479
Title: #479: MAN: Document that auth and access IPA and AD providers rely on id_provider
being set to the same type

Label: +Changes requested

[sssd PR#479][synchronized] MAN: Document that auth and access IPA and AD providers rely on id_provider being set to the same type

[jhrozek]

   URL: https://github.com/SSSD/sssd/pull/479
Author: jhrozek
 Title: #479: MAN: Document that auth and access IPA and AD providers rely on id_provider
being set to the same type
Action: synchronized

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

[sssd PR#479][comment] MAN: Document that auth and access IPA and AD providers rely on id_provider being set to the same type

[jhrozek]

  URL: https://github.com/SSSD/sssd/pull/479
Title: #479: MAN: Document that auth and access IPA and AD providers rely on id_provider
being set to the same type

jhrozek commented:
"""
I used @justin-stephenson's wording, thank you very much for the review
"""

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

[sssd PR#479][-Changes requested] MAN: Document that auth and access IPA and AD providers rely on id_provider being set to the same type

[jhrozek]

  URL: https://github.com/SSSD/sssd/pull/479
Title: #479: MAN: Document that auth and access IPA and AD providers rely on id_provider
being set to the same type

Label: -Changes requested

[sssd PR#479][comment] MAN: Document that auth and access IPA and AD providers rely on id_provider being set to the same type

[lslebodn]

  URL: https://github.com/SSSD/sssd/pull/479
Title: #479: MAN: Document that auth and access IPA and AD providers rely on id_provider
being set to the same type

lslebodn commented:
"""
master:
* 89ed594e8800fdb288248006e8867a077a57bc6f

sssd-1-14:
* 7a93d0f94855f1574a709b4c930ee99e6e71cb73

sssd-1-13:
* 06dd476f726e484854a85077804720b9a0f6b2da
"""

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

[sssd PR#479][closed] MAN: Document that auth and access IPA and AD providers rely on id_provider being set to the same type

[lslebodn]

   URL: https://github.com/SSSD/sssd/pull/479
Author: jhrozek
 Title: #479: MAN: Document that auth and access IPA and AD providers rely on id_provider
being set to the same type
Action: closed

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

[sssd PR#479][+Pushed] MAN: Document that auth and access IPA and AD providers rely on id_provider being set to the same type

[lslebodn]

  URL: https://github.com/SSSD/sssd/pull/479
Title: #479: MAN: Document that auth and access IPA and AD providers rely on id_provider
being set to the same type

Label: +Pushed