[SSSD] [sssd PR#109][comment] SSSCTL: fix netgroup-show parsing

2017-01-06 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/109 Title: #109: SSSCTL: fix netgroup-show parsing jhrozek commented: """ This patch is OK, but only for sssd-1-14. In master, we already fall back to parsing the name string as short name if parsing the qualified name fails. I’m not thrilled about that,

[SSSD] Re: [sssd PR#109][comment] SSSCTL: fix netgroup-show parsing

2017-01-06 Thread Jakub Hrozek
This patch is OK, but only for sssd-1-14. In master, we already fall back to parsing the name string as short name if parsing the qualified name fails. I’m not thrilled about that, because it can conceal legitimate errors, but it’s needed atm to use cache_req everywhere. So yeah, ack to this

[SSSD] [sssd PR#100][-Changes requested] Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds

2017-01-06 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/100 Title: #100: Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an

[SSSD] [sssd PR#100][+Pushed] Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds

2017-01-06 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/100 Title: #100: Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#100][closed] Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds

2017-01-06 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/100 Author: amitkumar50 Title: #100: Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/100/head:pr100 git

[SSSD] [sssd PR#100][comment] Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds

2017-01-06 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/100 Title: #100: Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds lslebodn commented: """ master: * dfbbe39f49ba3f7ff64370fecef971f091214d47 LS """ See the full comment at

[SSSD] [sssd PR#100][comment] Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds

2017-01-06 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/100 Title: #100: Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds lslebodn commented: """ Ups; I didn't notice that it's already done ACK++ """ See the full comment at

[SSSD] [sssd PR#110][comment] Add more DEBUG messages to help admins diagnose Kerberos login failures

2017-01-06 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/110 Title: #110: Add more DEBUG messages to help admins diagnose Kerberos login failures lslebodn commented: """ @mzidek-rh @sumit-bose Are you fine with this version? """ See the full comment at

[SSSD] [sssd PR#100][comment] Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds

2017-01-06 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/100 Title: #100: Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds lslebodn commented: """ @amitkumar50 Could you squash the patches to the single patch? """ See the full comment at

[SSSD] [sssd PR#113][comment] Adds an integration test for sssctl netgroup-show so that we don't regress again like we did in ticket #3267.

2017-01-06 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/113 Title: #113: Adds an integration test for sssctl netgroup-show so that we don't regress again like we did in ticket #3267. lslebodn commented: """ I think you can use prefix "test_sssctl" instead of tests. At the same time you can also mention

[SSSD] [sssd PR#113][+Changes requested] Adds an integration test for sssctl netgroup-show so that we don't regress again like we did in ticket #3267.

2017-01-06 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/113 Title: #113: Adds an integration test for sssctl netgroup-show so that we don't regress again like we did in ticket #3267. Label: +Changes requested ___ sssd-devel mailing list --

[SSSD] [sssd PR#109][comment] SSSCTL: fix netgroup-show parsing

2017-01-06 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/109 Title: #109: SSSCTL: fix netgroup-show parsing lslebodn commented: """ @jhrozek Have you already decided whether it would be better to have this patch or 7b293a5? """ See the full comment at

[SSSD] Re: [Freeipa-devel] [RFC] Matching and Mapping Certificates

2017-01-06 Thread Sumit Bose
On Fri, Jan 06, 2017 at 08:50:14AM +0100, Jan Cholasta wrote: > On 5.1.2017 10:39, Sumit Bose wrote: > > On Mon, Jan 02, 2017 at 09:18:47AM +0100, Jan Cholasta wrote: > > > On 18.10.2016 07:34, Jan Cholasta wrote: > > > > On 17.10.2016 16:50, Rob Crittenden wrote: > > > > > Jan Cholasta wrote: > >

[SSSD] [sssd PR#102][comment] Sssctl no case sensitive searches

2017-01-06 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/102 Title: #102: Sssctl no case sensitive searches lslebodn commented: """ On (15/12/16 01:46), Jakub Hrozek wrote: >* 35ecfab87a24031e55798b22975e02832ee0f2ad >* 715abb607540945cc82355e94712da7ac9746a67 >* d6e875c49d6be650a03fc14f00a680734b23ef66 >*