[SSSD] [sssd PR#467][comment] nss-idmap: allow NULL result in *_timeout calls

2017-12-08 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/467 Title: #467: nss-idmap: allow NULL result in *_timeout calls lslebodn commented: """ master: * bba068c535d23eebff61f592bddb3a6438446d6f """ See the full comment at https://github.com/SSSD/sssd/pull/467#issuecomment-350261699 __

[SSSD] [sssd PR#467][comment] nss-idmap: allow NULL result in *_timeout calls

2017-12-08 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/467 Title: #467: nss-idmap: allow NULL result in *_timeout calls fidencio commented: """ CI: http://vm-031.${abc}/logs/job/82/41/summary.html Please, whoever pushes this patch add also @abbra as reviewer. """ See the full comment at https://github.com/SS

[SSSD] [sssd PR#467][comment] nss-idmap: allow NULL result in *_timeout calls

2017-12-08 Thread abbra
URL: https://github.com/SSSD/sssd/pull/467 Title: #467: nss-idmap: allow NULL result in *_timeout calls abbra commented: """ I just did a test with slapi-nis that evicts users/groups from cache in case ID overrides did change and everything worked with this pull request while without it ns-sla

[SSSD] [sssd PR#467][comment] nss-idmap: allow NULL result in *_timeout calls

2017-12-08 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/467 Title: #467: nss-idmap: allow NULL result in *_timeout calls fidencio commented: """ I'm firing a CI build. Code-wise it looks good and **does** make sense to allow the result argument to be NULL as in some cases the caller is just not interested in t