[SSSD] [sssd PR#607][comment] Do not apply override_homedir and override_shell to files provider

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/607 Title: #607: Do not apply override_homedir and override_shell to files provider jhrozek commented: """ Thanks for the quick turnaround, now really the last thing, can you move the definition of is_files_provider from utils.c to domain_info_utils.c so t

[SSSD] [sssd PR#607][synchronized] Do not apply override_homedir and override_shell to files provider

2018-07-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/607 Author: fidencio Title: #607: Do not apply override_homedir and override_shell to files provider Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/607/head:pr607 git checkout pr

[SSSD] [sssd PR#607][comment] Do not apply override_homedir and override_shell to files provider

2018-07-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/607 Title: #607: Do not apply override_homedir and override_shell to files provider fidencio commented: """ Done! """ See the full comment at https://github.com/SSSD/sssd/pull/607#issuecomment-403755674 ___ sssd

[SSSD] [sssd PR#611][comment] Do not build the local provider by default

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/611 Title: #611: Do not build the local provider by default jhrozek commented: """ Thanks, this is a good start. We also want to remove the whole pysss.local interface and don't build the `sss_*` tools like `sss_useradd` and so on. If you want to remove t

[SSSD] [sssd PR#611][comment] Do not build the local provider by default

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/611 Title: #611: Do not build the local provider by default jhrozek commented: """ btw I'm not sold either way myself, on one hand the local provider might be handy for tests, on the other hand I haven't used it in a long time myself. So I would not be co

[SSSD] [sssd PR#611][comment] Do not build the local provider by default

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/611 Title: #611: Do not build the local provider by default jhrozek commented: """ About the sysdb tests, can you paste what issues you faced here? Or push a branch? I could see issues with not matching number of groups returned because the local provider

[SSSD] [sssd PR#607][+Accepted] Do not apply override_homedir and override_shell to files provider

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/607 Title: #607: Do not apply override_homedir and override_shell to files provider Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le..

[SSSD] [sssd PR#610][+Pushed] AD: consult the Partial Attribute Set instead of searching the GC for POSIX attributes

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/610 Title: #610: AD: consult the Partial Attribute Set instead of searching the GC for POSIX attributes Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email

[SSSD] [sssd PR#610][closed] AD: consult the Partial Attribute Set instead of searching the GC for POSIX attributes

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/610 Author: jhrozek Title: #610: AD: consult the Partial Attribute Set instead of searching the GC for POSIX attributes Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/610/head:pr610 g

[SSSD] [sssd PR#610][comment] AD: consult the Partial Attribute Set instead of searching the GC for POSIX attributes

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/610 Title: #610: AD: consult the Partial Attribute Set instead of searching the GC for POSIX attributes jhrozek commented: """ * master: * 5b2b6493dfb3c1f2cb945356e34c70d8c5d64185

[SSSD] [sssd PR#609][+Pushed] SUDO: Don't save duplicates when saving qualified names

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/609 Title: #609: SUDO: Don't save duplicates when saving qualified names Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedor

[SSSD] [sssd PR#609][comment] SUDO: Don't save duplicates when saving qualified names

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/609 Title: #609: SUDO: Don't save duplicates when saving qualified names jhrozek commented: """ * master: e75601bfe8475e4c1f821255a3f80c0a5d30f2be

[SSSD] [sssd PR#609][closed] SUDO: Don't save duplicates when saving qualified names

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/609 Author: jhrozek Title: #609: SUDO: Don't save duplicates when saving qualified names Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/609/head:pr609 git checkout pr609 __

[SSSD] [sssd PR#607][comment] Do not apply override_homedir and override_shell to files provider

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/607 Title: #607: Do not apply override_homedir and override_shell to files provider jhrozek commented: """ * master: 2373df99b9ff166bcbdb0c69fe5e28bd32ea43bf

[SSSD] [sssd PR#607][+Pushed] Do not apply override_homedir and override_shell to files provider

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/607 Title: #607: Do not apply override_homedir and override_shell to files provider Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@

[SSSD] [sssd PR#607][closed] Do not apply override_homedir and override_shell to files provider

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/607 Author: fidencio Title: #607: Do not apply override_homedir and override_shell to files provider Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/607/head:pr607 git checkout pr607 __

[SSSD] [sssd PR#613][synchronized] cache_req: Keep the files provider as the first domain to be searched

2018-07-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/613 Author: fidencio Title: #613: cache_req: Keep the files provider as the first domain to be searched Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/613/head:pr613 git checkou

[SSSD] [sssd PR#613][-Changes requested] cache_req: Keep the files provider as the first domain to be searched

2018-07-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/613 Title: #613: cache_req: Keep the files provider as the first domain to be searched Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to ss

[SSSD] [sssd PR#613][comment] cache_req: Keep the files provider as the first domain to be searched

2018-07-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/613 Title: #613: cache_req: Keep the files provider as the first domain to be searched fidencio commented: """ @sumit-bose, I've updated the patchset. Changes since last version: - rebased atop of git master; - making use of is_files_provider() call that

[SSSD] [sssd PR#613][+Changes requested] cache_req: Keep the files provider as the first domain to be searched

2018-07-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/613 Title: #613: cache_req: Keep the files provider as the first domain to be searched Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to ss

[SSSD] [sssd PR#613][comment] cache_req: Keep the files provider as the first domain to be searched

2018-07-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/613 Title: #613: cache_req: Keep the files provider as the first domain to be searched fidencio commented: """ And make-check-valgrind is failing. Let me add the "Changes Requested" label again. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#614][opened] nss_protocol_fill_initgr: skip incomplete groups instead of bailing out

2018-07-10 Thread asheplyakov
URL: https://github.com/SSSD/sssd/pull/614 Author: asheplyakov Title: #614: nss_protocol_fill_initgr: skip incomplete groups instead of bailing out Action: opened PR body: """ Suppose the user U is a member of (AD) groups D1\A, D1\B, D2\X, and no domain controllers in the domain D2 can be rea

[SSSD] [sssd PR#614][comment] nss_protocol_fill_initgr: skip incomplete groups instead of bailing out

2018-07-10 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/614 Title: #614: nss_protocol_fill_initgr: skip incomplete groups instead of bailing out centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/614#issuecomment-403867723 __

[SSSD] [sssd PR#614][comment] nss_protocol_fill_initgr: skip incomplete groups instead of bailing out

2018-07-10 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/614 Title: #614: nss_protocol_fill_initgr: skip incomplete groups instead of bailing out centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/614#issuecomment-403867722 __

[SSSD] [sssd PR#614][comment] nss_protocol_fill_initgr: skip incomplete groups instead of bailing out

2018-07-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/614 Title: #614: nss_protocol_fill_initgr: skip incomplete groups instead of bailing out fidencio commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/614#issuecomment-403871241 _

[SSSD] [sssd PR#595][+Accepted] sudo: allow use of default domain suffix (sssd-1-13)

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/595 Title: #595: sudo: allow use of default domain suffix (sssd-1-13) Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedora

[SSSD] [sssd PR#595][comment] sudo: allow use of default domain suffix (sssd-1-13)

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/595 Title: #595: sudo: allow use of default domain suffix (sssd-1-13) jhrozek commented: """ The tests passed, I just forgot to push the patch to sssd-1-13.. """ See the full comment at https://github.com/SSSD/sssd/pull/595#issuecomment-403873254

[SSSD] [sssd PR#595][comment] sudo: allow use of default domain suffix (sssd-1-13)

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/595 Title: #595: sudo: allow use of default domain suffix (sssd-1-13) jhrozek commented: """ * sssd-1-13: 5f234f231e5293a255fd9d864561d77091f5bcb6 """ See the full comment at https://github.com/SSSD/sssd/pull/595#issuecomment-403873819 ___

[SSSD] [sssd PR#595][+Pushed] sudo: allow use of default domain suffix (sssd-1-13)

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/595 Title: #595: sudo: allow use of default domain suffix (sssd-1-13) Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedoraho

[SSSD] [sssd PR#595][closed] sudo: allow use of default domain suffix (sssd-1-13)

2018-07-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/595 Author: pbrezina Title: #595: sudo: allow use of default domain suffix (sssd-1-13) Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/595/head:pr595 git checkout pr595

[SSSD] [sssd PR#611][comment] Do not build the local provider by default

2018-07-10 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/611 Title: #611: Do not build the local provider by default pbrezina commented: """ > Thanks, this is a good start. We also want to remove the whole pysss.local > interface and don't build the sss_* tools like sss_useradd and so on. > > If you want to remo

[SSSD] [sssd PR#612][comment] crypto: Silence a Coverity warning in sss_hmac_sha1()

2018-07-10 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/612 Title: #612: crypto: Silence a Coverity warning in sss_hmac_sha1() sumit-bose commented: """ Thanks, I'll do a CI an Coverity run before sending the ACK. """ See the full comment at https://github.com/SSSD/sssd/pull/612#issuecomment-403911646

[SSSD] [sssd PR#613][-Changes requested] cache_req: Keep the files provider as the first domain to be searched

2018-07-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/613 Title: #613: cache_req: Keep the files provider as the first domain to be searched Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to ss

[SSSD] [sssd PR#613][synchronized] cache_req: Keep the files provider as the first domain to be searched

2018-07-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/613 Author: fidencio Title: #613: cache_req: Keep the files provider as the first domain to be searched Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/613/head:pr613 git checkou

[SSSD] [sssd PR#613][comment] cache_req: Keep the files provider as the first domain to be searched

2018-07-10 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/613 Title: #613: cache_req: Keep the files provider as the first domain to be searched fidencio commented: """ @sumit-bose, I've updated the patchset. """ See the full comment at https://github.com/SSSD/sssd/pull/613#issuecomment-403964058 __