[SSSD] [sssd PR#390][comment] NSS: Add option to disable memcache

2017-09-29 Thread simo5
URL: https://github.com/SSSD/sssd/pull/390 Title: #390: NSS: Add option to disable memcache simo5 commented: """ @mzidek-rh yes I think in some cases we may want a much larger group memcache than user memcache """ See the full comment at https://github.com/SSSD/sssd/pull/390#issuecomment-3331

[SSSD] [sssd PR#275][comment] Implement access verification by rhost using ldap_access_order rhost option

2017-09-29 Thread akamensky
URL: https://github.com/SSSD/sssd/pull/275 Title: #275: Implement access verification by rhost using ldap_access_order rhost option akamensky commented: """ Added unit tests for the functionality (also squashed all commits to one). """ See the full comment at https://github.com/SSSD/sssd/pull

[SSSD] [sssd PR#390][comment] NSS: Add option to disable memcache

2017-09-29 Thread simo5
URL: https://github.com/SSSD/sssd/pull/390 Title: #390: NSS: Add option to disable memcache simo5 commented: """ @mzidek-rh yes I think in some cases we may want a much larger group memcache than user memcache """ See the full comment at https://github.com/SSSD/sssd/pull/390#issuecomment-3331

[SSSD] [sssd PR#275][synchronized] Implement access verification by rhost using ldap_access_order rhost option

2017-09-29 Thread akamensky
URL: https://github.com/SSSD/sssd/pull/275 Author: akamensky Title: #275: Implement access verification by rhost using ldap_access_order rhost option Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/275/head:pr275 git

[SSSD] [sssd PR#396][opened] CI: Add missing http-parser-devel dependency

2017-09-29 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/396 Author: fidencio Title: #396: CI: Add missing http-parser-devel dependency Action: opened PR body: """ As reported on #3526, running $ ./contrib/ci/run on CentOS 7 minimal installation ends up in failure on configure because ./contrib/ci/deps.sh does

[SSSD] [sssd PR#390][comment] NSS: Add option to disable memcache

2017-09-29 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/390 Title: #390: NSS: Add option to disable memcache mzidek-rh commented: """ I have one question to the implementation. I will add one option for all memcache types, do we also want to have separate options for different types of memcache? """ See the f

[SSSD] [sssd PR#390][comment] NSS: Add option to disable memcache

2017-09-29 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/390 Title: #390: NSS: Add option to disable memcache mzidek-rh commented: """ I agree with the proposal. Will update new version. """ See the full comment at https://github.com/SSSD/sssd/pull/390#issuecomment-333102308

[SSSD] [sssd PR#275][synchronized] Implement access verification by rhost using ldap_access_order rhost option

2017-09-29 Thread akamensky
URL: https://github.com/SSSD/sssd/pull/275 Author: akamensky Title: #275: Implement access verification by rhost using ldap_access_order rhost option Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/275/head:pr275 git

[SSSD] [sssd PR#390][comment] NSS: Add option to disable memcache

2017-09-29 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/390 Title: #390: NSS: Add option to disable memcache fidencio commented: """ Hmm. I like @sumit-bose and @simo5 idea. We just have to be crystal clear that setting the new option to zero will disable the memcache (or any really small value) and that will

[SSSD] [sssd PR#390][comment] NSS: Add option to disable memcache

2017-09-29 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/390 Title: #390: NSS: Add option to disable memcache sumit-bose commented: """ Although I'm fine with the current version of the patch I'd like to make a suggestion which might help to make this change less controversial. I recently discussed with Simo th

[SSSD] [sssd PR#390][comment] NSS: Add option to disable memcache

2017-09-29 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/390 Title: #390: NSS: Add option to disable memcache fidencio commented: """ Just fired a CI build with the last version of this patch and I'll add the "Accepted" label as soon as I have the results. Thanks for your contribution! """ See the full comment