[SSSD] [sssd PR#221][comment] sss_cert_pem_to_der: Accept certificate with data before header

2017-03-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/221 Title: #221: sss_cert_pem_to_der: Accept certificate with data before header jhrozek commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/221#issuecomment-290816333 ___ sss

[SSSD] [sssd PR#220][opened] Fix setting up [application] domains without inherit_from

2017-03-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/220 Author: jhrozek Title: #220: Fix setting up [application] domains without inherit_from Action: opened PR body: """ To test, just define an application domain as the only domain in the config file and do not use the inherit_from attribute. A more deta

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Lukas Slebodnik
On (31/03/17 16:17), Jakub Hrozek wrote: >On Fri, Mar 31, 2017 at 03:43:23PM +0200, Lukas Slebodnik wrote: >> And if somebody prepare POC with rendering htmls with PR on github >> then we can mirror repo there. > >This is not a blocker to mirroring. Why? My experience is that if something is not r

[SSSD] [sssd PR#193][comment] UTIL: Use max 15 characters for AD host UPN

2017-03-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/193 Title: #193: UTIL: Use max 15 characters for AD host UPN lslebodn commented: """ And link to green CI results due to issues with test_enumeration. http://sssd-ci.duckdns.org/logs/job/67/14/summary.html """ See the full comment at https://github.com/S

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Jakub Hrozek
On Fri, Mar 31, 2017 at 03:43:23PM +0200, Lukas Slebodnik wrote: > And if somebody prepare POC with rendering htmls with PR on github > then we can mirror repo there. This is not a blocker to mirroring. ___ sssd-devel mailing list -- sssd-devel@lists.fed

[SSSD] Re: Regarding sssd.conf syntax check, going thru dinglib

2017-03-31 Thread Lukas Slebodnik
On (30/03/17 22:00), amit kumar wrote: >Hello Lukas, > >Thanks for response, yes we have ticket >https://pagure.io/SSSD/sssd/issue/416 > >But my query was regarding the design *how we parse smb.conf using >ding-lib.* > Here is a desing page which describe ding-libs part. https://fedorahosted.org/ss

[SSSD] [sssd PR#202][comment] T3315 infopipe group users master

2017-03-31 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/202 Title: #202: T3315 infopipe group users master mzidek-rh commented: """ On 03/31/2017 03:44 PM, lslebodn wrote: > On (31/03/17 06:34), mzidek-rh wrote: >>Actually, I looking at it again I do not think these patches solve the > issue. The reproducer is n

[SSSD] [sssd PR#202][comment] T3315 infopipe group users master

2017-03-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/202 Title: #202: T3315 infopipe group users master lslebodn commented: """ On (31/03/17 06:34), mzidek-rh wrote: >Actually, I looking at it again I do not think these patches solve the issue. >The reproducer is not correct. After you update all the members

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Lukas Slebodnik
On (31/03/17 15:23), Jakub Hrozek wrote: >On Fri, Mar 31, 2017 at 11:50:15AM +0200, Lukas Slebodnik wrote: >> On (31/03/17 11:00), Pavel Březina wrote: >> >On 03/31/2017 10:35 AM, Lukas Slebodnik wrote: >> >> On (30/03/17 21:18), Jakub Hrozek wrote: >> >> > Hi, >> >> > >> >> > I'd like to remove h

[SSSD] [sssd PR#202][comment] T3315 infopipe group users master

2017-03-31 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/202 Title: #202: T3315 infopipe group users master mzidek-rh commented: """ Actually, I looking at it again I do not think these patches solve the issue. The reproducer is not correct. After you update all the members in the group with dbus call in your r

[SSSD] [sssd PR#202][comment] T3315 infopipe group users master

2017-03-31 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/202 Title: #202: T3315 infopipe group users master mzidek-rh commented: """ Actually, I looking at it again I do not think these patches solve the issue. The reproducer is not correct. After you update all the members in the group with dbus call in your r

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Lukas Slebodnik
On (31/03/17 15:16), Jakub Hrozek wrote: >On Fri, Mar 31, 2017 at 10:55:29AM +0200, Sumit Bose wrote: >> On Fri, Mar 31, 2017 at 10:35:20AM +0200, Lukas Slebodnik wrote: >> > On (30/03/17 21:18), Jakub Hrozek wrote: >> > >Hi, >> > > >> > >I'd like to remove https://github.com/SSSD/gh-mailinglist-no

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Jakub Hrozek
On Fri, Mar 31, 2017 at 11:50:15AM +0200, Lukas Slebodnik wrote: > On (31/03/17 11:00), Pavel Březina wrote: > >On 03/31/2017 10:35 AM, Lukas Slebodnik wrote: > >> On (30/03/17 21:18), Jakub Hrozek wrote: > >> > Hi, > >> > > >> > I'd like to remove https://github.com/SSSD/gh-mailinglist-notificati

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Jakub Hrozek
On Thu, Mar 30, 2017 at 09:18:39PM +0200, Jakub Hrozek wrote: > Hi, > > I'd like to remove https://github.com/SSSD/gh-mailinglist-notifications > -- we're not currently using it, but we're rather using Martin Basti's > mail notifier. Deleted ___ sssd-de

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Jakub Hrozek
On Fri, Mar 31, 2017 at 10:55:29AM +0200, Sumit Bose wrote: > On Fri, Mar 31, 2017 at 10:35:20AM +0200, Lukas Slebodnik wrote: > > On (30/03/17 21:18), Jakub Hrozek wrote: > > >Hi, > > > > > >I'd like to remove https://github.com/SSSD/gh-mailinglist-notifications > > >-- we're not currently using i

[SSSD] Re: OK to just push converted docs from fedorahosted wiki to the pagure docs repo?

2017-03-31 Thread Jakub Hrozek
On Fri, Mar 31, 2017 at 10:53:03AM +0200, Lukas Slebodnik wrote: > On (30/03/17 21:22), Jakub Hrozek wrote: > >Hi, > > > >would anyone complain if I just push converted content from fedorahosted > >to the docs repo w/o review as long as there are no changes OR the > >changes are just alingnment of

[SSSD] [sssd PR#211][comment] IFP: Fix of limit = 0 (unlimited result)

2017-03-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/211 Title: #211: IFP: Fix of limit = 0 (unlimited result) lslebodn commented: """ retest this please """ See the full comment at https://github.com/SSSD/sssd/pull/211#issuecomment-290705474 ___ sssd-devel mailin

[SSSD] [sssd PR#208][comment] IFP: Filter with * in Users.ListByName method

2017-03-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/208 Title: #208: IFP: Filter with * in Users.ListByName method lslebodn commented: """ Could you rebase this PR? """ See the full comment at https://github.com/SSSD/sssd/pull/208#issuecomment-290705242 ___ sssd-

[SSSD] [sssd PR#202][comment] T3315 infopipe group users master

2017-03-31 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/202 Title: #202: T3315 infopipe group users master mzidek-rh commented: """ These patches fixed the issue for me. LGTM, if CI passes I will give you an ACK. """ See the full comment at https://github.com/SSSD/sssd/pull/202#issuecomment-290704624 _

[SSSD] Re: Fragile test enumeration

2017-03-31 Thread Lukas Slebodnik
On (31/03/17 11:58), Lukas Slebodnik wrote: >ehlo, > >we pushed recently many patches and I noticed increased amount of failures >in enumeration test. > >http://sssd-ci.duckdns.org/logs/job/66/82/summary.html >http://sssd-ci.duckdns.org/logs/job/66/83/summary.html >http://sssd-ci.duckdns.org/logs/j

[SSSD] [sssd PR#219][comment] BUILD: Fix compilation of libsss_certmap with libcrypto

2017-03-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/219 Title: #219: BUILD: Fix compilation of libsss_certmap with libcrypto lslebodn commented: """ retest this please """ See the full comment at https://github.com/SSSD/sssd/pull/219#issuecomment-290699839 ___ ss

[SSSD] [sssd PR#193][comment] UTIL: Use max 15 characters for AD host UPN

2017-03-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/193 Title: #193: UTIL: Use max 15 characters for AD host UPN lslebodn commented: """ Thank you very much for links. """ See the full comment at https://github.com/SSSD/sssd/pull/193#issuecomment-290695949 ___ ss

[SSSD] [sssd PR#193][-Changes requested] UTIL: Use max 15 characters for AD host UPN

2017-03-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/193 Title: #193: UTIL: Use max 15 characters for AD host UPN Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedora

[SSSD] [sssd PR#193][synchronized] UTIL: Use max 15 characters for AD host UPN

2017-03-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/193 Author: lslebodn Title: #193: UTIL: Use max 15 characters for AD host UPN Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/193/head:pr193 git checkout pr193 From f6dc8a0a50090c

[SSSD] [sssd PR#219][opened] BUILD: Fix compilation of libsss_certmap with libcrypto

2017-03-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/219 Author: lslebodn Title: #219: BUILD: Fix compilation of libsss_certmap with libcrypto Action: opened PR body: """ CC src/lib/certmap/libsss_certmap_la-sss_cert_content_nss.lo src/lib/certmap/sss_cert_content_nss.c:25:18:

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Pavel Březina
On 03/31/2017 12:16 PM, Lukas Slebodnik wrote: On (31/03/17 12:05), Pavel Březina wrote: On 03/31/2017 11:50 AM, Lukas Slebodnik wrote: On (31/03/17 11:00), Pavel Březina wrote: On 03/31/2017 10:35 AM, Lukas Slebodnik wrote: On (30/03/17 21:18), Jakub Hrozek wrote: Hi, I'd like to remove ht

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Lukas Slebodnik
On (31/03/17 12:05), Pavel Březina wrote: >On 03/31/2017 11:50 AM, Lukas Slebodnik wrote: >> On (31/03/17 11:00), Pavel Březina wrote: >> > On 03/31/2017 10:35 AM, Lukas Slebodnik wrote: >> > > On (30/03/17 21:18), Jakub Hrozek wrote: >> > > > Hi, >> > > > >> > > > I'd like to remove https://githu

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Pavel Březina
On 03/31/2017 11:50 AM, Lukas Slebodnik wrote: On (31/03/17 11:00), Pavel Březina wrote: On 03/31/2017 10:35 AM, Lukas Slebodnik wrote: On (30/03/17 21:18), Jakub Hrozek wrote: Hi, I'd like to remove https://github.com/SSSD/gh-mailinglist-notifications -- we're not currently using it, but we'

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Lukas Slebodnik
On (31/03/17 11:50), Lukas Slebodnik wrote: >On (31/03/17 11:00), Pavel Březina wrote: >>On 03/31/2017 10:35 AM, Lukas Slebodnik wrote: >>> On (30/03/17 21:18), Jakub Hrozek wrote: >>> > Hi, >>> > >>> > I'd like to remove https://github.com/SSSD/gh-mailinglist-notifications >>> > -- we're not curr

[SSSD] [sssd PR#218][comment] TEST: Adding krb5-libs to dependencies

2017-03-31 Thread celestian
URL: https://github.com/SSSD/sssd/pull/218 Title: #218: TEST: Adding krb5-libs to dependencies celestian commented: """ @lslebodn thanks. I think I see it now. New patch pushed. """ See the full comment at https://github.com/SSSD/sssd/pull/218#issuecomment-290671916 ___

[SSSD] [sssd PR#218][synchronized] TEST: Adding krb5-libs to dependencies

2017-03-31 Thread celestian
URL: https://github.com/SSSD/sssd/pull/218 Author: celestian Title: #218: TEST: Adding krb5-libs to dependencies Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/218/head:pr218 git checkout pr218 From 09426863f867732d74

[SSSD] Fragile test enumeration

2017-03-31 Thread Lukas Slebodnik
ehlo, we pushed recently many patches and I noticed increased amount of failures in enumeration test. http://sssd-ci.duckdns.org/logs/job/66/82/summary.html http://sssd-ci.duckdns.org/logs/job/66/83/summary.html http://sssd-ci.duckdns.org/logs/job/66/84/summary.html http://sssd-ci.duckdns.org/log

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Lukas Slebodnik
On (31/03/17 11:00), Pavel Březina wrote: >On 03/31/2017 10:35 AM, Lukas Slebodnik wrote: >> On (30/03/17 21:18), Jakub Hrozek wrote: >> > Hi, >> > >> > I'd like to remove https://github.com/SSSD/gh-mailinglist-notifications >> > -- we're not currently using it, but we're rather using Martin Basti

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Pavel Březina
On 03/31/2017 10:35 AM, Lukas Slebodnik wrote: On (30/03/17 21:18), Jakub Hrozek wrote: Hi, I'd like to remove https://github.com/SSSD/gh-mailinglist-notifications -- we're not currently using it, but we're rather using Martin Basti's mail notifier. +1 At the same time, I'd like to add a mi

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Sumit Bose
On Fri, Mar 31, 2017 at 10:35:20AM +0200, Lukas Slebodnik wrote: > On (30/03/17 21:18), Jakub Hrozek wrote: > >Hi, > > > >I'd like to remove https://github.com/SSSD/gh-mailinglist-notifications > >-- we're not currently using it, but we're rather using Martin Basti's > >mail notifier. > > > +1 > >

[SSSD] Re: OK to just push converted docs from fedorahosted wiki to the pagure docs repo?

2017-03-31 Thread Lukas Slebodnik
On (30/03/17 21:22), Jakub Hrozek wrote: >Hi, > >would anyone complain if I just push converted content from fedorahosted >to the docs repo w/o review as long as there are no changes OR the >changes are just alingnment of 1.15 design documents with the actual >implementation? > Design documents can

[SSSD] Re: Remove https://github.com/SSSD/gh-mailinglist-notifications, add https://github.com/SSSD/docs

2017-03-31 Thread Lukas Slebodnik
On (30/03/17 21:18), Jakub Hrozek wrote: >Hi, > >I'd like to remove https://github.com/SSSD/gh-mailinglist-notifications >-- we're not currently using it, but we're rather using Martin Basti's >mail notifier. > +1 >At the same time, I'd like to add a mirror of >https://pagure.io/SSSD/docs to githu

[SSSD] [sssd PR#218][comment] TEST: Adding krb5-libs to dependencies

2017-03-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/218 Title: #218: TEST: Adding krb5-libs to dependencies lslebodn commented: """ I think that ticket 3353 is mainly about following change ``` commit 0700118d8388c38b8cb28279510b206b76a3a411 Author: Jakub Hrozek Date: Tue Dec 13 17:17:16 2016 +0100 T

[SSSD] [sssd PR#218][comment] TEST: Adding krb5-libs to dependencies

2017-03-31 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/218 Title: #218: TEST: Adding krb5-libs to dependencies lslebodn commented: """ This change does not fix anything. Fedora/el{6,7} already install kr5 in spec file `BuildRequires: krb5-devel` """ See the full comment at https://github.com/SSSD/sssd/pull/2