[SSSD] [sssd PR#565][-Accepted] CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/565 Title: #565: CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send

[SSSD] [sssd PR#565][+Pushed] CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/565 Title: #565: CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send

[SSSD] [sssd PR#565][closed] CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/565 Author: jhrozek Title: #565: CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd

[SSSD] [sssd PR#565][comment] CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/565 Title: #565: CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up fidencio commented: """ master: 2952de7 """ See the full comment at https://github.com/SSSD/sssd/pull/565#issuecomment-386421306

[SSSD] [sssd PR#564][-Accepted] Do not keep allocating external groups on a long-lived context

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/564 Title: #564: Do not keep allocating external groups on a long-lived context Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#564][closed] Do not keep allocating external groups on a long-lived context

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/564 Author: jhrozek Title: #564: Do not keep allocating external groups on a long-lived context Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/564/head:pr564 git checkout pr564

[SSSD] [sssd PR#563][+Pushed] SYSDB: Only check non-POSIX groups for GID conflicts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/563 Title: #563: SYSDB: Only check non-POSIX groups for GID conflicts Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#564][comment] Do not keep allocating external groups on a long-lived context

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/564 Title: #564: Do not keep allocating external groups on a long-lived context fidencio commented: """ master: 10213ef """ See the full comment at https://github.com/SSSD/sssd/pull/564#issuecomment-386420892

[SSSD] [sssd PR#563][-Accepted] SYSDB: Only check non-POSIX groups for GID conflicts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/563 Title: #563: SYSDB: Only check non-POSIX groups for GID conflicts Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#563][closed] SYSDB: Only check non-POSIX groups for GID conflicts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/563 Author: jhrozek Title: #563: SYSDB: Only check non-POSIX groups for GID conflicts Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/563/head:pr563 git checkout pr563

[SSSD] [sssd PR#563][comment] SYSDB: Only check non-POSIX groups for GID conflicts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/563 Title: #563: SYSDB: Only check non-POSIX groups for GID conflicts fidencio commented: """ master: 8a8285c """ See the full comment at https://github.com/SSSD/sssd/pull/563#issuecomment-386420712 ___

[SSSD] [sssd PR#562][+Pushed] AD: Warn if the LDAP schema is overriden with the AD provider

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/562 Title: #562: AD: Warn if the LDAP schema is overriden with the AD provider Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#562][closed] AD: Warn if the LDAP schema is overriden with the AD provider

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/562 Author: jhrozek Title: #562: AD: Warn if the LDAP schema is overriden with the AD provider Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/562/head:pr562 git checkout pr562

[SSSD] [sssd PR#562][-Accepted] AD: Warn if the LDAP schema is overriden with the AD provider

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/562 Title: #562: AD: Warn if the LDAP schema is overriden with the AD provider Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#562][comment] AD: Warn if the LDAP schema is overriden with the AD provider

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/562 Title: #562: AD: Warn if the LDAP schema is overriden with the AD provider fidencio commented: """ master: 3cff2c5 """ See the full comment at https://github.com/SSSD/sssd/pull/562#issuecomment-386420347

[SSSD] [sssd PR#561][-Accepted] DYNDNS: Retry also on timeouts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/561 Title: #561: DYNDNS: Retry also on timeouts Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#561][closed] DYNDNS: Retry also on timeouts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/561 Author: jhrozek Title: #561: DYNDNS: Retry also on timeouts Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/561/head:pr561 git checkout pr561

[SSSD] [sssd PR#561][comment] DYNDNS: Retry also on timeouts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/561 Title: #561: DYNDNS: Retry also on timeouts fidencio commented: """ master: 65034a7 b57dfac """ See the full comment at https://github.com/SSSD/sssd/pull/561#issuecomment-386419595 ___ sssd-devel mailing

[SSSD] [sssd PR#562][+Accepted] AD: Warn if the LDAP schema is overriden with the AD provider

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/562 Title: #562: AD: Warn if the LDAP schema is overriden with the AD provider Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#562][comment] AD: Warn if the LDAP schema is overriden with the AD provider

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/562 Title: #562: AD: Warn if the LDAP schema is overriden with the AD provider fidencio commented: """ Ack! http://vm-031.${abc}/logs/job/88/46/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/562#issuecomment-386414765

[SSSD] [sssd PR#563][+Accepted] SYSDB: Only check non-POSIX groups for GID conflicts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/563 Title: #563: SYSDB: Only check non-POSIX groups for GID conflicts Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#561][comment] DYNDNS: Retry also on timeouts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/561 Title: #561: DYNDNS: Retry also on timeouts fidencio commented: """ Ack! CI: http://vm-031.${abc}/logs/job/88/42/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/561#issuecomment-386266504

[SSSD] [sssd PR#564][comment] Do not keep allocating external groups on a long-lived context

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/564 Title: #564: Do not keep allocating external groups on a long-lived context fidencio commented: """ Ack! CI: http://vm-031.${abc}/logs/job/88/43/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/564#issuecomment-386280572

[SSSD] [sssd PR#565][comment] CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/565 Title: #565: CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up fidencio commented: """ CI: http://vm-031.${abc}/logs/job/88/44/summary.html test_ipa_subdom_server failed and I've ran it a bunch of times

[SSSD] [sssd PR#565][+Accepted] CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/565 Title: #565: CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send

[SSSD] handling whitespaces in netgroup triples

2018-05-03 Thread Josef Cejka
Hi, I have noticed that sssd does not trim whitespaces from strings while parsing netgroup triples. Comment inside code explains that it follow the nss_ldap implementation: src/db/sysdb_search.c: 1687 /* This function splits a three-tuple into three strings 1688  * It assumes that any whitespace

[SSSD] [sssd PR#564][comment] Do not keep allocating external groups on a long-lived context

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/564 Title: #564: Do not keep allocating external groups on a long-lived context fidencio commented: """ Ack! CI: http://vm-031.abc.idm.lab.eng.brq.redhat.com/logs/job/88/43/summary.html """ See the full comment at

[SSSD] [sssd PR#564][+Accepted] Do not keep allocating external groups on a long-lived context

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/564 Title: #564: Do not keep allocating external groups on a long-lived context Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#562][comment] AD: Warn if the LDAP schema is overriden with the AD provider

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/562 Title: #562: AD: Warn if the LDAP schema is overriden with the AD provider fidencio commented: """ Right. I'm firing the CI and I'll add the accepted label as soon as I get the results. """ See the full comment at

[SSSD] [sssd PR#561][comment] DYNDNS: Retry also on timeouts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/561 Title: #561: DYNDNS: Retry also on timeouts fidencio commented: """ Ack! CI: http://vm-031.abc.idm.lab.eng.brq.redhat.com/logs/job/88/42/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/561#issuecomment-386266504

[SSSD] [sssd PR#561][+Accepted] DYNDNS: Retry also on timeouts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/561 Title: #561: DYNDNS: Retry also on timeouts Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#564][comment] Do not keep allocating external groups on a long-lived context

2018-05-03 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/564 Title: #564: Do not keep allocating external groups on a long-lived context jhrozek commented: """ > On 3 May 2018, at 09:41, fidencio wrote: > > In situations like this I really would like to (if @sumit-bose agrees) to

[SSSD] [sssd PR#564][comment] Do not keep allocating external groups on a long-lived context

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/564 Title: #564: Do not keep allocating external groups on a long-lived context fidencio commented: """ In situations like this I really would like to (if @sumit-bose agrees) to add a Signed-off-by: Sumit ... Is that okay for you, @jhrozek? """ See the

[SSSD] [sssd PR#564][comment] Do not keep allocating external groups on a long-lived context

2018-05-03 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/564 Title: #564: Do not keep allocating external groups on a long-lived context jhrozek commented: """ > On 3 May 2018, at 09:07, fidencio wrote: > > Patch looks good. I'm running the internal CI just for the sake of the >

[SSSD] [sssd PR#563][comment] SYSDB: Only check non-POSIX groups for GID conflicts

2018-05-03 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/563 Title: #563: SYSDB: Only check non-POSIX groups for GID conflicts jhrozek commented: """ > On 3 May 2018, at 09:09, fidencio wrote: > > Hmm. The non-POSIX case just didn't come to my mind when writing the original > patch

[SSSD] [sssd PR#562][comment] AD: Warn if the LDAP schema is overriden with the AD provider

2018-05-03 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/562 Title: #562: AD: Warn if the LDAP schema is overriden with the AD provider jhrozek commented: """ > On 3 May 2018, at 09:10, fidencio wrote: > > Do we have a note in the manual about this? If not, together with the warning

[SSSD] [sssd PR#561][comment] DYNDNS: Retry also on timeouts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/561 Title: #561: DYNDNS: Retry also on timeouts fidencio commented: """ Patches look good. I'm running the internal CI just for the sake of the process and I'll add the accepted label as soon as I get the results. """ See the full comment at

[SSSD] [sssd PR#562][comment] AD: Warn if the LDAP schema is overriden with the AD provider

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/562 Title: #562: AD: Warn if the LDAP schema is overriden with the AD provider fidencio commented: """ I'm not adding the "Changes Requested" label now because so far I'm not requesting any changes. """ See the full comment at

[SSSD] [sssd PR#562][comment] AD: Warn if the LDAP schema is overriden with the AD provider

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/562 Title: #562: AD: Warn if the LDAP schema is overriden with the AD provider fidencio commented: """ Do we have a note in the manual about this? If not, together with the warning I'd suggest to also add it there. """ See the full comment at

[SSSD] [sssd PR#563][comment] SYSDB: Only check non-POSIX groups for GID conflicts

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/563 Title: #563: SYSDB: Only check non-POSIX groups for GID conflicts fidencio commented: """ Hmm. The non-POSIX case just didn't come to my mind when writing the original patch down. The patch looks good and I'm running the internal CI just for the sake

[SSSD] [sssd PR#564][comment] Do not keep allocating external groups on a long-lived context

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/564 Title: #564: Do not keep allocating external groups on a long-lived context fidencio commented: """ Patch looks good. I'm running the internal CI just for the sake of the process and I'll add the accepted label as soon as I get the results. """ See

[SSSD] [sssd PR#565][comment] CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up

2018-05-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/565 Title: #565: CACHE_REQ: Do not fail the domain locator plugin if ID outside the domain range is looked up fidencio commented: """ Patch looks good. I'm running the internal CI just for the sake of the process and I'll add the accepted label as soon