[SSSD] [sssd PR#46][comment] sss_client: Defer thread cancellation until completion of nss/pam operations

2016-10-28 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/46 Title: #46: sss_client: Defer thread cancellation until completion of nss/pam operations sumit-bose commented: """ The patch looks good and works as expected and removes the dependency to libpthread. If I understand the it correctly there is a small

[SSSD] [sssd PR#68][comment] MAN: Document different defaults for AD provider

2016-10-28 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/68 Title: #68: MAN: Document different defaults for AD provider sumit-bose commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/68#issuecomment-256867038 ___ sssd-devel

[SSSD] [sssd PR#60][comment] Document ad_access_filter search for nested groups

2016-10-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/60 Title: #60: Document ad_access_filter search for nested groups jhrozek commented: """ On Thu, Oct 27, 2016 at 08:48:46AM -0700, taupehat wrote: > @jhrozek I did not intend to remove anything from the last iteration. It's > possible that I used the

[SSSD] [sssd PR#57][comment] LDAP/AD: resolve domain local groups for remote users

2016-10-28 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/57 Title: #57: LDAP/AD: resolve domain local groups for remote users jhrozek commented: """ CI: http://sssd-ci.duckdns.org/logs/job/56/05/summary.html Coverity also didn't find any errors. For some reason, re-running just the tests that failed previously