URL: https://github.com/SSSD/sssd/pull/522
Title: #522: Prepare SSSD to support IPA in trust to Samba AD
jhrozek commented:
"""
* master:
14faec9cd9437ef116ae054412d25ec2e820e409
0648053a7c99de5148c4cffea68c47a3f660303d
"""
See the full comment at
URL: https://github.com/SSSD/sssd/pull/522
Title: #522: Prepare SSSD to support IPA in trust to Samba AD
jhrozek commented:
"""
coverity and CI did not find any issues -> +Accepted
"""
See the full comment at
https://github.com/SSSD/sssd/pull/522#issuecomment-405346482
URL: https://github.com/SSSD/sssd/pull/522
Title: #522: Prepare SSSD to support IPA in trust to Samba AD
abbra commented:
"""
@jhrozek ping, is it possible to review this pull request?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/522#issuecomment-403373886
URL: https://github.com/SSSD/sssd/pull/522
Title: #522: Prepare SSSD to support IPA in trust to Samba AD
abbra commented:
"""
@jhrozek, @sumit-bose I've updated this pull request with a new code that adds
a missing `cn=trusts,$SUFFIX` base after a user base is parsed. I find this
approach
URL: https://github.com/SSSD/sssd/pull/522
Title: #522: Prepare SSSD to support IPA in trust to Samba AD
abbra commented:
"""
@jhrozek @sumit-bose I've updated the patches to address Jakub's comments.TDO
lookup should only be performed in the server mode, so I added that. At the
server side
URL: https://github.com/SSSD/sssd/pull/522
Title: #522: Prepare SSSD to support IPA in trust to Samba AD
jhrozek commented:
"""
I only scrolled through the patches, I must admit I didn't do any testing yet.
For the search bases, I would just appreciate a more descriptive variable names
than
URL: https://github.com/SSSD/sssd/pull/522
Title: #522: Prepare SSSD to support IPA in trust to Samba AD
sumit-bose commented:
"""
ok to test
"""
See the full comment at
https://github.com/SSSD/sssd/pull/522#issuecomment-367938353
___
sssd-devel
URL: https://github.com/SSSD/sssd/pull/522
Title: #522: Prepare SSSD to support IPA in trust to Samba AD
centos-ci commented:
"""
Can one of the admins verify this patch?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/522#issuecomment-367925811
URL: https://github.com/SSSD/sssd/pull/522
Title: #522: Prepare SSSD to support IPA in trust to Samba AD
centos-ci commented:
"""
Can one of the admins verify this patch?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/522#issuecomment-367925813