[SSSD] [sssd PR#13][-Accepted] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#13][closed] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/13 Author: celestian Title: #13: MEMBEROF: Don't resolve members if they are removed Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/13/head:pr13 git checkout pr13

[SSSD] [sssd PR#13][+Pushed] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed jhrozek commented: """ master: e0903f41922721edf292a9f7e6605a4519db53a1 eaf44bc07dda469a20be07d46737d93f518e2047 """ See the full comment at

[SSSD] [sssd PR#44][-Accepted] libwbclient-sssd: update interface to version 0.13

2016-10-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/44 Title: #44: libwbclient-sssd: update interface to version 0.13 Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#44][closed] libwbclient-sssd: update interface to version 0.13

2016-10-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/44 Author: sumit-bose Title: #44: libwbclient-sssd: update interface to version 0.13 Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/44/head:pr44 git checkout pr44

[SSSD] [sssd PR#44][comment] libwbclient-sssd: update interface to version 0.13

2016-10-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/44 Title: #44: libwbclient-sssd: update interface to version 0.13 lslebodn commented: """ master: * f3347a0c72afc75b4d829e9981d1bac6b05a8306 LS """ See the full comment at https://github.com/SSSD/sssd/pull/44#issuecomment-253857506

[SSSD] [sssd PR#47][comment] BUILD: Fix build without /sbin/service installed on the build host

2016-10-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/47 Title: #47: BUILD: Fix build without /sbin/service installed on the build host lslebodn commented: """ master: * a2485c56319041f0021a46d63aac38ec2a5a6b2e * 0d52311adc48ecbe45e84c42332dece12c6d34fe * a3cf63eab87fc34bd871c7bab0676c3a06558011 *

[SSSD] [sssd PR#47][comment] BUILD: Fix build without /sbin/service installed on the build host

2016-10-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/47 Title: #47: BUILD: Fix build without /sbin/service installed on the build host lslebodn commented: """ On (14/10/16 06:42), Jakub Hrozek wrote: >let's try again.. > Finally, ACK :-) http://sssd-ci.duckdns.org/logs/job/55/09/summary.html And I would

[SSSD] [sssd PR#48][comment] sssctl: Flags for commadn initialization

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/48 Title: #48: sssctl: Flags for commadn initialization jhrozek commented: """ On Fri, Oct 14, 2016 at 08:03:20AM -0700, mzidek-rh wrote: > I see the comment did not get forwarded to the devel list, so pasting again: > > > Sorry, I am out of ideas here.

[SSSD] [sssd PR#48][comment] sssctl: Flags for commadn initialization

2016-10-14 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/48 Title: #48: sssctl: Flags for commadn initialization mzidek-rh commented: """ I see the comment did not get forwarded to the devel list, so pasting again: Sorry, I am out of ideas here. What name do you propose? Or will it be enough if I just add a

[SSSD] [sssd PR#48][+Changes requested] sssctl: Flags for commadn initialization

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/48 Title: #48: sssctl: Flags for commadn initialization Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#47][synchronized] BUILD: Fix build without /sbin/service installed on the build host

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/47 Author: jhrozek Title: #47: BUILD: Fix build without /sbin/service installed on the build host Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/47/head:pr47 git checkout pr47

[SSSD] [sssd PR#47][comment] BUILD: Fix build without /sbin/service installed on the build host

2016-10-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/47 Title: #47: BUILD: Fix build without /sbin/service installed on the build host lslebodn commented: """ I hope you noticed that CI build failed: https://ci.centos.org/job/sssd-CentOS6/87/ and https://ci.centos.org/job/sssd-CentOS7/100/ ``` error:

[SSSD] [sssd PR#43][comment] RESPONDER: Enable sudoRule in case insen. domains​ (1.14)

2016-10-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.14) celestian commented: """ So, I set ```case_sensitive = preserving``` and: ``` ssh -l TEsT@scorpion.domain 192.168.122.65 TEsT@scorpion.domain@192.168.122.65's password:

[SSSD] [sssd PR#34][comment] cache_req: move from switch to plugins

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/34 Title: #34: cache_req: move from switch to plugins jhrozek commented: """ Actually, sorry, one more Coverity warning: ``` Error: FORWARD_NULL (CWE-476): sssd-1.14.2/src/responder/common/cache_req/cache_req_search.c:105: var_compare_op: Comparing

[SSSD] [sssd PR#34][+Changes requested] cache_req: move from switch to plugins

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/34 Title: #34: cache_req: move from switch to plugins Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#43][comment] RESPONDER: Enable sudoRule in case insen. domains​ (1.14)

2016-10-14 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.14) pbrezina commented: """ User that is stored in AD as teST can have multiple login names since case sensitivity is ignored. It can login as test, but also as teST, TEst, Test, ...

[SSSD] [sssd PR#43][comment] RESPONDER: Enable sudoRule in case insen. domains​ (1.14)

2016-10-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.14) celestian commented: """ I added user 'teST' on AD. His login on Linux box is 'test'. Patch works. Or missed I something? """ See the full comment at

[SSSD] [sssd PR#47][-Changes requested] BUILD: Fix build without /sbin/service installed on the build host

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/47 Title: #47: BUILD: Fix build without /sbin/service installed on the build host Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#47][synchronized] BUILD: Fix build without /sbin/service installed on the build host

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/47 Author: jhrozek Title: #47: BUILD: Fix build without /sbin/service installed on the build host Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/47/head:pr47 git checkout pr47

[SSSD] [sssd PR#47][+Changes requested] BUILD: Fix build without /sbin/service installed on the build host

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/47 Title: #47: BUILD: Fix build without /sbin/service installed on the build host Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#34][-Changes requested] cache_req: move from switch to plugins

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/34 Title: #34: cache_req: move from switch to plugins Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#43][comment] RESPONDER: Enable sudoRule in case insen. domains​ (1.14)

2016-10-14 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.14) pbrezina commented: """ I didn't do a thorough review but we also need to search with lover case values in responder so also names like "admiNISTRATOR" will match on case insensitive

[SSSD] [sssd PR#43][comment] RESPONDER: Enable sudoRule in case insen. domains​ (1.14)

2016-10-14 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/43 Title: #43: RESPONDER: Enable sudoRule in case insen. domains​ (1.14) pbrezina commented: """ I didn't do a thorough review but we also need to search with lover case values in responder so also names like "admiNISTRATOR" will match on case insensitive

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed celestian commented: """ So, ldap patch passed intg. tests locally. """ See the full comment at https://github.com/SSSD/sssd/pull/13#issuecomment-253782152

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed jhrozek commented: """ yes, if we don't know how to trigger the code, we shouldn't push it. OK to push the two commits now? """ See the full comment at

[SSSD] [sssd PR#34][+Changes requested] cache_req: move from switch to plugins

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/34 Title: #34: cache_req: move from switch to plugins Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#34][comment] cache_req: move from switch to plugins

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/34 Title: #34: cache_req: move from switch to plugins jhrozek commented: """ Apart from the nitpicks, the code looks better to me (I like that the lookup functionality is abstracted, so adding new lookups will be easy). I tested several kinds of lookups,

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed lslebodn commented: """ On (14/10/16 04:48), celestian wrote: >I did manual testing with reproducer above. And I ran chmake (it is without >intg., isn't it). >Now I check ldap patch with

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed celestian commented: """ I did manual testing with reproducer above. And I ran chmake (it is without intg., isn't it). Now I check ldap patch with intg. """ See the full comment at

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed lslebodn commented: """ On (14/10/16 04:33), Jakub Hrozek wrote: >btw since the memberof patch we couldn't test is gone, I'm fine with pushing >these two. > I would like to know Petr's

[SSSD] [sssd PR#48][-Changes requested] sssctl: Flags for commadn initialization

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/48 Title: #48: sssctl: Flags for commadn initialization Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed jhrozek commented: """ btw since the memberof patch we couldn't test is gone, I'm fine with pushing these two. """ See the full comment at

[SSSD] [sssd PR#13][+Accepted] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#48][synchronized] sssctl: Flags for commadn initialization

2016-10-14 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/48 Author: mzidek-rh Title: #48: sssctl: Flags for commadn initialization Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/48/head:pr48 git checkout pr48 From

[SSSD] [sssd PR#48][comment] sssctl: Flags for commadn initialization

2016-10-14 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/48 Title: #48: sssctl: Flags for commadn initialization mzidek-rh commented: """ Ok, new patch uploaded. """ See the full comment at https://github.com/SSSD/sssd/pull/48#issuecomment-253774056 ___ sssd-devel

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed lslebodn commented: """ > Great, the result is the same. Both patches could fix it. So we can use just > ldap patch. What do you mean by both patches could fix it? I aplied just a

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed lslebodn commented: """ > Great, the result is the same. Both patches could fix it. So we can use just > ldap patch. What do you mean by both patches could fix it? I aplied just a

[SSSD] [sssd PR#48][comment] sssctl: Flags for commadn initialization

2016-10-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/48 Title: #48: sssctl: Flags for commadn initialization lslebodn commented: """ IMHO, it would be nicer if you add new macro SSS_TOOL_COMMAND_FLAGS (or different name) rather then changing usage of SSS_TOOL_COMMAND and SSS_TOOL_COMMAND_NOMSG on all

[SSSD] [sssd PR#48][+Changes requested] sssctl: Flags for commadn initialization

2016-10-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/48 Title: #48: sssctl: Flags for commadn initialization Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] Re: controls for solaris clients for password-less login (sshkey)

2016-10-14 Thread Jakub Hrozek
On Fri, Oct 14, 2016 at 11:36:17AM +0200, Csaba Dobo wrote: > Hi, > I need to add the below control to openldap: > 1.3.6.1.4.1.42.2.27.9.5.8 (Account Usability Control) > > Anyone knows how to do such thing? The openldap-technical mailing list:

[SSSD] controls for solaris clients for password-less login (sshkey)

2016-10-14 Thread Csaba Dobo
Hi, I need to add the below control to openldap: 1.3.6.1.4.1.42.2.27.9.5.8 (Account Usability Control) Anyone knows how to do such thing? thnx Csaba ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#47][synchronized] BUILD: Fix build without /sbin/service installed on the build host

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/47 Author: jhrozek Title: #47: BUILD: Fix build without /sbin/service installed on the build host Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/47/head:pr47 git checkout pr47

[SSSD] [sssd PR#47][comment] BUILD: Fix build without /sbin/service installed on the build host

2016-10-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/47 Title: #47: BUILD: Fix build without /sbin/service installed on the build host jhrozek commented: """ @lslebodn suggested on IRC to only search for service in /sbin and /usr/sbin. I added another patch that does that.. CI:

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed celestian commented: """ Great, the result is the same. Both patches could fix it. So we can use just ldap patch. """ See the full comment at

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed celestian commented: """ Oh no, I tested it for ldap, not for ipa provider. Wait a minute a will test it again. :-( """ See the full comment at

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed celestian commented: """ I changed the author of ldap patch to @sumit-bose. """ See the full comment at https://github.com/SSSD/sssd/pull/13#issuecomment-253734041

[SSSD] [sssd PR#13][synchronized] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/13 Author: celestian Title: #13: MEMBEROF: Don't resolve members if they are removed Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/13/head:pr13 git checkout pr13 From

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed fidencio commented: """ On Fri, Oct 14, 2016 at 9:46 AM, celestian wrote: > So, I looked this patch set again. Both, ldap and memberof patch, can fix > the issue

[SSSD] [sssd PR#13][comment] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed celestian commented: """ So, I looked this patch set again. Both, ldap and memberof patch, can fix the issue itself. I removed memberof patch. Original author of ldap patch is @sumit-bose

[SSSD] [sssd PR#13][-Changes requested] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#13][synchronized] MEMBEROF: Don't resolve members if they are removed

2016-10-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/13 Author: celestian Title: #13: MEMBEROF: Don't resolve members if they are removed Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/13/head:pr13 git checkout pr13 From