[SSSD] [sssd PR#5] Miscellanous patches for the sssd-secrets responder (comment)

2016-08-30 Thread jhrozek
jhrozek commented on a pull request """ @pbrezina can you add the ack label (at least we'd try if you have the permissions to do so..) """ See the full comment at https://github.com/SSSD/sssd/pull/5#issuecomment-243407276

[SSSD] [sssd PR#5] Miscellanous patches for the sssd-secrets responder (closed)

2016-08-30 Thread jhrozek
jhrozek's pull request #5: "Miscellanous patches for the sssd-secrets responder" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/5 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5/head:pr5 git checkout pr5 _

[SSSD] [sssd PR#5] Miscellanous patches for the sssd-secrets responder (+pushed)

2016-08-30 Thread jhrozek
jhrozek's pull request #5: "Miscellanous patches for the sssd-secrets responder" label *pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/5 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.f

[SSSD] [sssd PR#5] Miscellanous patches for the sssd-secrets responder (-ack)

2016-08-30 Thread jhrozek
jhrozek's pull request #5: "Miscellanous patches for the sssd-secrets responder" label *ack* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/5 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fe

[SSSD] [sssd PR#6] Applications should never define _USE_GNU themselves, but rely on _GNU_SOURCE (opened)

2016-08-30 Thread jhrozek
jhrozek's pull request #6: "Applications should never define _USE_GNU themselves, but rely on _GNU_SOURCE" was opened PR body: """ None """ See the full pull-request at https://github.com/SSSD/sssd/pull/6 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fe

[SSSD] [sssd PR#6] Applications should never define _USE_GNU themselves, but rely on _GNU_SOURCE (comment)

2016-08-30 Thread jhrozek
jhrozek commented on a pull request """ * master: 1384d0ce6ea741aefb56b0006b6268d76e6cc2c2 """ See the full comment at https://github.com/SSSD/sssd/pull/6#issuecomment-243453157 ___ sssd-devel mailing list sssd-deve

[SSSD] [sssd PR#6] Applications should never define _USE_GNU themselves, but rely on _GNU_SOURCE (-ack)

2016-08-30 Thread jhrozek
jhrozek's pull request #6: "Applications should never define _USE_GNU themselves, but rely on _GNU_SOURCE" label *ack* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/6 ___ sssd-devel mailing list sssd-devel@lists.fedorah

[SSSD] [sssd PR#6] Applications should never define _USE_GNU themselves, but rely on _GNU_SOURCE (+pushed)

2016-08-30 Thread jhrozek
jhrozek's pull request #6: "Applications should never define _USE_GNU themselves, but rely on _GNU_SOURCE" label *pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/6 ___ sssd-devel mailing list sssd-devel@lists.fedora

[SSSD] [sssd PR#6] Applications should never define _USE_GNU themselves, but rely on _GNU_SOURCE (closed)

2016-08-30 Thread jhrozek
jhrozek's pull request #6: "Applications should never define _USE_GNU themselves, but rely on _GNU_SOURCE" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/6 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/6/head:

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-30 Thread jhrozek
jhrozek commented on a pull request """ On Tue, Aug 30, 2016 at 11:47:09AM -0700, lslebodn wrote: > Please provide a test-case (probably a hierarchy of groups) I was able to reproduce with: $ ipa group-show group20 Group name: group20 GID: 935600011 Member groups

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-30 Thread jhrozek
jhrozek commented on a pull request """ OK, also looking at gdb confirms this is the right fix of course: (gdb) frame 0 #0 sdap_initgr_nested_get_membership_diff (mem_ctx=0x1176190, sysdb=0x1120040, opts=0x1134d90, dom=0x110b610, group=0x1184b80, all_groups=0x1184a80,

[SSSD] [sssd PR#7] Fix initgroups with nested groups (+ack)

2016-08-30 Thread jhrozek
sumit-bose's pull request #7: "Fix initgroups with nested groups" label *ack* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/7 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/ad

[SSSD] [sssd PR#8] MAN: Document the ldap_user_gid_number option (opened)

2016-08-31 Thread jhrozek
jhrozek's pull request #8: "MAN: Document the ldap_user_gid_number option" was opened PR body: """ Some users, mostly those who cannot join the Linux machines to their AD domain resort to using the LDAP provider with ID mapping. But in order for that to work correctly, the ldap_user_gid_number

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-31 Thread jhrozek
jhrozek commented on a pull request """ On Wed, Aug 31, 2016 at 12:36:37AM -0700, sumit-bose wrote: > On Tue, Aug 30, 2016 at 12:36:20PM -0700, Jakub Hrozek wrote: > > On Tue, Aug 30, 2016 at 11:47:09AM -0700, lslebodn wrote: > > > > About the discussi

[SSSD] [sssd PR#9] SYSDB: Removing of unused parametr (+ack)

2016-08-31 Thread jhrozek
celestian's pull request #9: "SYSDB: Removing of unused parametr" label *ack* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/9 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/ad

[SSSD] [sssd PR#9] SYSDB: Removing of unused parametr (comment)

2016-08-31 Thread jhrozek
jhrozek commented on a pull request """ ACK, CI pending """ See the full comment at https://github.com/SSSD/sssd/pull/9#issuecomment-243902393 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://list

[SSSD] [sssd PR#8] MAN: Document the ldap_user_gid_number option (comment)

2016-09-01 Thread jhrozek
jhrozek commented on a pull request """ CI: http://sssd-ci.duckdns.org/logs/job/52/87/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/8#issuecomment-244052362 ___ sssd-devel mailing list sss

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-09-01 Thread jhrozek
jhrozek commented on a pull request """ On Wed, Aug 31, 2016 at 07:15:10AM -0700, lslebodn wrote: > On (31/08/16 01:47), sumit-bose wrote: > >On Wed, Aug 31, 2016 at 01:30:12AM -0700, Jakub Hrozek wrote: > >> On Wed, Aug 31, 2016 at 12:36:37AM -0700, sumit-bose w

[SSSD] [sssd PR#8] MAN: Document the ldap_user_gid_number option (closed)

2016-09-01 Thread jhrozek
jhrozek's pull request #8: "MAN: Document the ldap_user_gid_number option" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/8 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/8/head:pr8 git checkout pr8 ___

[SSSD] [sssd PR#8] MAN: Document the ldap_user_gid_number option (comment)

2016-09-01 Thread jhrozek
jhrozek commented on a pull request """ * master: 6f59bb822d1e54e178207be45e382f4ee173c434 """ See the full comment at https://github.com/SSSD/sssd/pull/8#issuecomment-244054299 ___ sssd-devel mailing list sssd-deve

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-09-01 Thread jhrozek
jhrozek commented on a pull request """ * master: 5bd3bef4a655fdfacd2f5df8a2343fe7bc68a771 """ See the full comment at https://github.com/SSSD/sssd/pull/7#issuecomment-244058146 ___ sssd-devel mailing list sssd-deve

[SSSD] [sssd PR#7] Fix initgroups with nested groups (closed)

2016-09-01 Thread jhrozek
sumit-bose's pull request #7: "Fix initgroups with nested groups" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/7 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/7/head:pr7 git checkout pr7 _

[SSSD] [sssd PR#8] MAN: Document the ldap_user_gid_number option (+pushed)

2016-09-01 Thread jhrozek
jhrozek's pull request #8: "MAN: Document the ldap_user_gid_number option" label *pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/8 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorah

[SSSD] [sssd PR#7] Fix initgroups with nested groups (+pushed)

2016-09-01 Thread jhrozek
sumit-bose's pull request #7: "Fix initgroups with nested groups" label *pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/7 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org

[SSSD] [sssd PR#9] SYSDB: Removing of unused parametr (+pushed)

2016-09-01 Thread jhrozek
celestian's pull request #9: "SYSDB: Removing of unused parametr" label *pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/9 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org

[SSSD] [sssd PR#9] SYSDB: Removing of unused parametr (comment)

2016-09-01 Thread jhrozek
jhrozek commented on a pull request """ * master: 0d1d9d8001232f74eca63cbba6c400d507b33823 """ See the full comment at https://github.com/SSSD/sssd/pull/9#issuecomment-244058945 ___ sssd-devel mailing list sssd-deve

[SSSD] [sssd PR#9] SYSDB: Removing of unused parametr (closed)

2016-09-01 Thread jhrozek
celestian's pull request #9: "SYSDB: Removing of unused parametr" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/9 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/9/head:pr9 git checkout pr9 _

[SSSD] [sssd PR#14] Attempting to create a qualified name from a NULL pointer broke sss_groupadd (opened)

2016-09-06 Thread jhrozek
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL pointer broke sss_groupadd" was opened PR body: """ The log said: (Tue Sep 6 15:18:31:935105 2016) [sss_groupadd] [sysdb_timestamp_cache_connect] (0x0400): No timestamp cache for LOCAL (Tue Sep 6 15:18:31:935255 201

[SSSD] [sssd PR#14] Attempting to create a qualified name from a NULL pointer broke sss_groupadd (synchronize)

2016-09-06 Thread jhrozek
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL pointer broke sss_groupadd" was synchronize See the full pull-request at https://github.com/SSSD/sssd/pull/14 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/1

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (opened)

2016-09-06 Thread jhrozek
jhrozek's pull request #15: "Avoid returning System Error on clock skew" was opened PR body: """ Rather return ERR_NETWORK_IO and at least let the user authenticate offline. """ See the full pull-request at https://github.com/SSSD/sssd/pull/15 ... or pull the PR as Git branch: git remote add ghs

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (comment)

2016-09-06 Thread jhrozek
jhrozek commented on a pull request """ Thanks for the ack, I would also like to ask @sumit-bose if he agrees with the change. """ See the full comment at https://github.com/SSSD/sssd/pull/15#issuecomment-244927163 __

[SSSD] [sssd PR#14] Attempting to create a qualified name from a NULL pointer broke sss_groupadd (+nack)

2016-09-06 Thread jhrozek
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL pointer broke sss_groupadd" label *nack* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/14 ___ sssd-devel mailing list sssd-devel@lists.fedorah

[SSSD] [sssd PR#14] Attempting to create a qualified name from a NULL pointer broke sss_groupadd (comment)

2016-09-06 Thread jhrozek
jhrozek commented on a pull request """ On Tue, Sep 06, 2016 at 04:54:29AM -0700, lslebodn wrote: > On (06/09/16 03:14), Jakub Hrozek wrote: > >The log said: > >(Tue Sep 6 15:18:31:935105 2016) [sss_groupadd] > >[sysdb_timestamp_cache_connect] (0x0400): No t

[SSSD] [sssd PR#10] UTIL: Remove support to libini older than 1.0.0 (comment)

2016-09-06 Thread jhrozek
jhrozek commented on a pull request """ On Thu, Sep 01, 2016 at 09:01:54AM -0700, lslebodn wrote: > On (01/09/16 08:35), fidencio wrote: > >On Thu, Sep 1, 2016 at 2:10 PM, lslebodn wrote: > > > >> On (01/09/16 04:32), Jakub Hrozek wrote: > >> >

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

2016-09-06 Thread jhrozek
jhrozek commented on a pull request """ On Fri, Sep 02, 2016 at 08:37:07AM -0700, celestian wrote: > I wrote the conditions in fully way for clarity. Yes, it is possible to write > it in condensed way. > > My question in note is about which type of our CI I can use.

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

2016-09-06 Thread jhrozek
celestian's pull request #13: "MEMBEROF: Don't resolve members if they are removed" label *nack* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/13 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (comment)

2016-09-06 Thread jhrozek
jhrozek commented on a pull request """ On Tue, Sep 06, 2016 at 05:10:07AM -0700, Pavel Březina wrote: > On 09/06/2016 01:51 PM, Jakub Hrozek wrote: > > Thanks for the ack, I would also like to ask @sumit-bose > > <https://github.com/sumit-bose> if he agrees w

[SSSD] [sssd PR#10] UTIL: Remove support to libini older than 1.0.0 (comment)

2016-09-06 Thread jhrozek
jhrozek commented on a pull request """ On Tue, Sep 06, 2016 at 05:16:00AM -0700, fidencio wrote: > The distributions that would break with this patch are: > - RHEL/CentOS 5 and older I don't think we care about RHEL-5 with master and I'm not sure sssd master

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (+nack)

2016-09-06 Thread jhrozek
jhrozek's pull request #15: "Avoid returning System Error on clock skew" label *nack* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/15 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahost

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (comment)

2016-09-06 Thread jhrozek
jhrozek commented on a pull request """ good idea """ See the full comment at https://github.com/SSSD/sssd/pull/15#issuecomment-244944667 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fed

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (-ack)

2016-09-06 Thread jhrozek
jhrozek's pull request #15: "Avoid returning System Error on clock skew" label *ack* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/15 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahos

[SSSD] [sssd PR#16] TOOLS: sss_groupshow did not work (+nack)

2016-09-06 Thread jhrozek
mzidek-rh's pull request #16: "TOOLS: sss_groupshow did not work" label *nack* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/16 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/

[SSSD] [sssd PR#10] UTIL: Remove support to libini older than 1.0.0 (+deferred)

2016-09-06 Thread jhrozek
fidencio's pull request #10: "UTIL: Remove support to libini older than 1.0.0" label *deferred* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/10 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.

[SSSD] [sssd PR#10] UTIL: Remove support to libini older than 1.0.0 (comment)

2016-09-07 Thread jhrozek
jhrozek commented on a pull request """ On Tue, Sep 06, 2016 at 11:49:09AM -0700, lslebodn wrote: > IMHO, it might be better to close this PR. > If we decide to dor support for libini_config < 1.1 or 1.2 > then it will be a different patch anyway. @see my previous

[SSSD] [sssd PR#14] Attempting to create a qualified name from a NULL pointer broke sss_groupadd (+rejected)

2016-09-07 Thread jhrozek
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL pointer broke sss_groupadd" label *rejected* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/14 ___ sssd-devel mailing list sssd-devel@lists.fed

[SSSD] [sssd PR#14] Attempting to create a qualified name from a NULL pointer broke sss_groupadd (comment)

2016-09-07 Thread jhrozek
jhrozek commented on a pull request """ On Tue, Sep 06, 2016 at 10:23:13AM -0700, mzidek-rh wrote: > The PR#16 includes jhrozek's patch from this PR as well as CI test + fix for > sss_groupshow. OK, thank you, I will close this one as rejected (just so that we don

[SSSD] [sssd PR#14] Attempting to create a qualified name from a NULL pointer broke sss_groupadd (-Changes requested)

2016-09-07 Thread jhrozek
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL pointer broke sss_groupadd" label *Changes requested* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/14 ___ sssd-devel mailing list sssd-deve

[SSSD] [sssd PR#14] Attempting to create a qualified name from a NULL pointer broke sss_groupadd (-rejected)

2016-09-07 Thread jhrozek
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL pointer broke sss_groupadd" label *rejected* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/14 ___ sssd-devel mailing list sssd-devel@lists.f

[SSSD] [sssd PR#14] Attempting to create a qualified name from a NULL pointer broke sss_groupadd (+rejected)

2016-09-07 Thread jhrozek
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL pointer broke sss_groupadd" label *rejected* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/14 ___ sssd-devel mailing list sssd-devel@lists.fed

[SSSD] [sssd PR#14] Attempting to create a qualified name from a NULL pointer broke sss_groupadd (closed)

2016-09-07 Thread jhrozek
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL pointer broke sss_groupadd" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/14 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/14/hea

[SSSD] [sssd PR#19] KRB5: Send the output username, not internal fqname to krb5_child (opened)

2016-09-07 Thread jhrozek
jhrozek's pull request #19: "KRB5: Send the output username, not internal fqname to krb5_child" was opened PR body: """ Resolves: https://fedorahosted.org/sssd/ticket/3172 krb5_child calls krb5_kuserok() during the access phase which checks if a particular user is allowed to authenticate as

[SSSD] [sssd PR#19] KRB5: Send the output username, not internal fqname to krb5_child (comment)

2016-09-07 Thread jhrozek
jhrozek commented on a pull request """ btw feel free to ping me on RH IRC for a link that shows the patch fixes the RH tests.. """ See the full comment at https://github.com/SSSD/sssd/pull/19#issuecomment-245325705 __

[SSSD] [sssd PR#19] KRB5: Send the output username, not internal fqname to krb5_child (+Accepted)

2016-09-07 Thread jhrozek
jhrozek's pull request #19: "KRB5: Send the output username, not internal fqname to krb5_child" label *Accepted* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/19 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.o

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (comment)

2016-09-07 Thread jhrozek
jhrozek commented on a pull request """ On Tue, Sep 06, 2016 at 06:09:58AM -0700, Jakub Hrozek wrote: > good idea ah, only when I started to implement this I realized it's already done :) See: https://github.com/SSSD/sssd/blob/master/src/providers/krb5/krb5_child

[SSSD] [sssd PR#18] [PATCHES] sss_user/groupmod fixes (closed)

2016-09-11 Thread jhrozek
mzidek-rh's pull request #18: "[PATCHES] sss_user/groupmod fixes" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/18 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/18/head:pr18 git checkout pr18

[SSSD] [sssd PR#19] KRB5: Send the output username, not internal fqname to krb5_child (closed)

2016-09-11 Thread jhrozek
jhrozek's pull request #19: "KRB5: Send the output username, not internal fqname to krb5_child" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/19 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/19/head:pr19 git

[SSSD] [sssd PR#19] KRB5: Send the output username, not internal fqname to krb5_child (+Pushed)

2016-09-11 Thread jhrozek
jhrozek's pull request #19: "KRB5: Send the output username, not internal fqname to krb5_child" label *Pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/19 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#19] KRB5: Send the output username, not internal fqname to krb5_child (-Accepted)

2016-09-11 Thread jhrozek
jhrozek's pull request #19: "KRB5: Send the output username, not internal fqname to krb5_child" label *Accepted* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/19 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted

[SSSD] [sssd PR#11] SECRETS: Don't remove a container when it has children (+Changes requested)

2016-09-11 Thread jhrozek
fidencio's pull request #11: "SECRETS: Don't remove a container when it has children" label *Changes requested* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/11 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.or

[SSSD] [sssd PR#16] TOOLS: sss_groupshow did not work (closed)

2016-09-12 Thread jhrozek
mzidek-rh's pull request #16: "TOOLS: sss_groupshow did not work" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/16 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/16/head:pr16 git checkout pr16 _

[SSSD] [sssd PR#17] Improve support for gdm Smartcard support (+Accepted)

2016-09-12 Thread jhrozek
sumit-bose's pull request #17: "Improve support for gdm Smartcard support" label *Accepted* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/17 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedo

[SSSD] [sssd PR#17] Improve support for gdm Smartcard support (comment)

2016-09-12 Thread jhrozek
jhrozek commented on a pull request """ I only read the code, haven't done any testing, really. I will push once CI finishes. """ See the full comment at https://github.com/SSSD/sssd/pull/17#issuecomment-246290312 _

[SSSD] [sssd PR#22] In commit c420ce830ac0b0b288a2a887ec2cfce5c748018c we changed a way how we (opened)

2016-09-12 Thread jhrozek
jhrozek's pull request #22: "In commit c420ce830ac0b0b288a2a887ec2cfce5c748018c we changed a way how we" was opened PR body: """ 'try a next server' in order to fix https://fedorahosted.org/sssd/ticket/3009. But since this change, results that return adminlimit exceeded (11) also mark the connec

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (edited)

2016-09-12 Thread jhrozek
jhrozek's pull request #22: "LDAP: Return partial results from adminlimit exceeded" was edited See the full pull-request at https://github.com/SSSD/sssd/pull/22 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/22/head:pr22 git checkout pr2

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (edited)

2016-09-12 Thread jhrozek
jhrozek's pull request #22: "LDAP: Return partial results from adminlimit exceeded" was edited See the full pull-request at https://github.com/SSSD/sssd/pull/22 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/22/head:pr22 git checkout pr2

[SSSD] [sssd PR#23] sss_groupshow does not work with mpg (+Pushed)

2016-09-13 Thread jhrozek
mzidek-rh's pull request #23: "sss_groupshow does not work with mpg" label *Pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/23 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted

[SSSD] [sssd PR#23] sss_groupshow does not work with mpg (closed)

2016-09-13 Thread jhrozek
mzidek-rh's pull request #23: "sss_groupshow does not work with mpg" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/23 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/23/head:pr23 git checkout pr23 __

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (+Pushed)

2016-09-13 Thread jhrozek
jhrozek's pull request #15: "Avoid returning System Error on clock skew" label *Pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/15 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedoraho

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (-Accepted)

2016-09-13 Thread jhrozek
jhrozek's pull request #15: "Avoid returning System Error on clock skew" label *Accepted* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/15 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedo

[SSSD] [sssd PR#15] Avoid returning System Error on clock skew (closed)

2016-09-13 Thread jhrozek
jhrozek's pull request #15: "Avoid returning System Error on clock skew" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/15 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/15/head:pr15 git checkout pr15 _

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (comment)

2016-09-14 Thread jhrozek
jhrozek commented on a pull request """ On Tue, Sep 13, 2016 at 01:33:49AM -0700, mzidek-rh wrote: > Makes sense, but I would prefer to add new "else if" branch with > ADMINLIMIT_EXCEEDED specific debug message. > > It would also make sense to change th

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (+Pushed)

2016-09-14 Thread jhrozek
jhrozek's pull request #22: "LDAP: Return partial results from adminlimit exceeded" label *Pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/22 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lis

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (-Accepted)

2016-09-14 Thread jhrozek
jhrozek's pull request #22: "LDAP: Return partial results from adminlimit exceeded" label *Accepted* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/22 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https:/

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (closed)

2016-09-14 Thread jhrozek
jhrozek's pull request #22: "LDAP: Return partial results from adminlimit exceeded" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/22 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/22/head:pr22 git checkout pr2

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (comment)

2016-09-14 Thread jhrozek
jhrozek commented on a pull request """ * master: 3319d964721396c07daba383ded6aaaf33ed6e3b """ See the full comment at https://github.com/SSSD/sssd/pull/22#issuecomment-246954357 ___ sssd-devel mailing list sssd-deve

[SSSD] [sssd PR#24] MAN: Add a manpage for the sssd-secrets responder (opened)

2016-09-14 Thread jhrozek
jhrozek's pull request #24: "MAN: Add a manpage for the sssd-secrets responder" was opened PR body: """ The manpage lists the options and adds API examples. """ See the full pull-request at https://github.com/SSSD/sssd/pull/24 ... or pull the PR as Git branch: git remote add ghsssd https://githu

[SSSD] [sssd PR#25] TESTS: Add integration tests for the proxy provider of sssd-secrets (opened)

2016-09-14 Thread jhrozek
jhrozek's pull request #25: "TESTS: Add integration tests for the proxy provider of sssd-secrets" was opened PR body: """ Execrcies the basic operations of the sssd-secrets responder and can be used as a basis to add more tests. """ See the full pull-request at https://github.com/SSSD/sssd/pull/

[SSSD] [sssd PR#11] SECRETS: Don't remove a container when it has children (comment)

2016-09-14 Thread jhrozek
jhrozek commented on a pull request """ The first patch was already pushed. The second looks good to me visually and I wrote a simple patch to verify the fix: https://github.com/jhrozek/sssd/commit/374eff89472a3181548495ed74deadaa690e921e """ See the full comme

[SSSD] [sssd PR#11] SECRETS: Don't remove a container when it has children (+Accepted)

2016-09-14 Thread jhrozek
fidencio's pull request #11: "SECRETS: Don't remove a container when it has children" label *Accepted* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/11 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https:/

[SSSD] [sssd PR#20] sss_override fails to export (closed)

2016-09-14 Thread jhrozek
mzidek-rh's pull request #20: "sss_override fails to export" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/20 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/20/head:pr20 git checkout pr20 __

[SSSD] [sssd PR#25] TESTS: Add integration tests for the proxy provider of sssd-secrets (comment)

2016-09-14 Thread jhrozek
jhrozek commented on a pull request """ btw here are CI tests on all supported systems: http://sssd-ci.duckdns.org/logs/job/53/41/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/25#issuecomment-246998696

[SSSD] [sssd PR#26] KRB5: Fixing FQ name of user in krb5_setup() (comment)

2016-09-14 Thread jhrozek
jhrozek commented on a pull request """ Why not do it the other way around and qualify all names in the mapping? That needs to be done only once unlike this approach that needs to unqualify the names every time they are used. """ See the full comment at https:

[SSSD] [sssd PR#26] KRB5: Fixing FQ name of user in krb5_setup() (+Changes requested)

2016-09-14 Thread jhrozek
celestian's pull request #26: "KRB5: Fixing FQ name of user in krb5_setup()" label *Changes requested* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/26 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https:/

[SSSD] [sssd PR#24][comment] MAN: Add a manpage for the sssd-secrets responder

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/24 Title: #24: MAN: Add a manpage for the sssd-secrets responder jhrozek commented: """ it would be nice if @simo5 or @tiran could skim over the manpage """ See the full comment at https://github.com/SSSD/sssd/

[SSSD] [sssd PR#17][comment] Improve support for gdm Smartcard support

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/17 Title: #17: Improve support for gdm Smartcard support jhrozek commented: """ OK, even the added patch looks good to me and I just sent the patches to CI to be sure we don't break anything. """ See the full comment at

[SSSD] [sssd PR#24][comment] MAN: Add a manpage for the sssd-secrets responder

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/24 Title: #24: MAN: Add a manpage for the sssd-secrets responder jhrozek commented: """ On Fri, Sep 16, 2016 at 05:03:04AM -0700, Christian Heimes wrote: > * Can SSSD's secret responder contact Custodia over Unix socket? I'

[SSSD] [sssd PR#25][-Changes requested] TESTS: Add integration tests for the proxy provider of sssd-secrets

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/25 Title: #25: TESTS: Add integration tests for the proxy provider of sssd-secrets Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-de

[SSSD] [sssd PR#25][synchronized] TESTS: Add integration tests for the proxy provider of sssd-secrets

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/25 Author: jhrozek Title: #25: TESTS: Add integration tests for the proxy provider of sssd-secrets Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/25/head:pr25 git checkout pr25

[SSSD] [sssd PR#26][+Changes requested] KRB5: Fixing FQ name of user in krb5_setup()

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/26 Title: #26: KRB5: Fixing FQ name of user in krb5_setup() Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorah

[SSSD] [sssd PR#26][comment] KRB5: Fixing FQ name of user in krb5_setup()

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/26 Title: #26: KRB5: Fixing FQ name of user in krb5_setup() jhrozek commented: """ You also need to use the output name for `kr->kuserok_user`, currently it's using qualified name, which is not going to work (see the branch just bel

[SSSD] [sssd PR#17][comment] Improve support for gdm Smartcard support

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/17 Title: #17: Improve support for gdm Smartcard support jhrozek commented: """ CI: http://sssd-ci.duckdns.org/logs/job/53/47/summary.html """ See the full comment at https://github.com/SSSD/sssd/

[SSSD] [sssd PR#17][+Pushed] Improve support for gdm Smartcard support

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/17 Title: #17: Improve support for gdm Smartcard support Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#17][closed] Improve support for gdm Smartcard support

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/17 Author: sumit-bose Title: #17: Improve support for gdm Smartcard support Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/17/head:pr17 git checkout pr17 __

[SSSD] [sssd PR#17][-Accepted] Improve support for gdm Smartcard support

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/17 Title: #17: Improve support for gdm Smartcard support 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#17][comment] Improve support for gdm Smartcard support

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/17 Title: #17: Improve support for gdm Smartcard support jhrozek commented: """ master: 35ba922bc51416f02877b53a6f25c04104ae5f03 3649b959709f1ab187092f054d4aace0798c98fa 71cd9f98150577224559bdc12c53c01ce6f2c3d9 ""&

[SSSD] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus jhrozek commented: """ I also wonder if the SIDs are needed. I don't have anything to add them in principle, except I would prefer to not add more attributes to the b

[SSSD] [sssd PR#11][-Accepted] SECRETS: Don't remove a container when it has children

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/11 Title: #11: SECRETS: Don't remove a container when it has children 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#11][comment] SECRETS: Don't remove a container when it has children

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/11 Title: #11: SECRETS: Don't remove a container when it has children jhrozek commented: """ * master: ab7b33fd7d820688545d5994a402cedf4bcdb6e1 """ See the full comment at https://github.com/SSSD/

[SSSD] [sssd PR#11][closed] SECRETS: Don't remove a container when it has children

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/11 Author: fidencio Title: #11: SECRETS: Don't remove a container when it has children Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/11/head:pr11 git checkout pr11 ___

[SSSD] [sssd PR#11][+Pushed] SECRETS: Don't remove a container when it has children

2016-09-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/11 Title: #11: SECRETS: Don't remove a container when it has children Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedoraho

  1   2   3   4   5   6   7   8   9   10   >