[SSSD] Re: [PATCH] Remove double semicolon at the end of line

2016-09-21 Thread Lukas Slebodnik
On (21/09/16 13:42), Pavel Březina wrote: >On 09/21/2016 11:00 AM, Pavel Březina wrote: >> On 09/20/2016 03:36 PM, Lukas Slebodnik wrote: >> > ehlo, >> > >> > simple patch attached + shell based unit test. >> > >> > LS >> >> Ack. > >You have typo in: > >+# Look for lines with with double

[SSSD] [sssd PR#31][opened] nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME

2016-09-21 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/31 Author: sumit-bose Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME Action: opened PR body: """ When adding support for UPNs, email addresses and aliases the SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME request were

[SSSD] [PATCH] SSSDConfig: Do not fail with nonexisting domains/services

2016-09-21 Thread Lukas Slebodnik
ehlo, Almost oneliner i there would not be a unit test :-) LS From b1cc72ab549c8134ea91a562070e95be0857654c Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Wed, 21 Sep 2016 13:56:43 +0200 Subject: [PATCH] SSSDConfig: Do not fail with nonexisting domains/services

[SSSD] [sssd PR#27][-Changes requested] Minor code changes

2016-09-21 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/27 Title: #27: Minor code changes Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] Re: [sssd PR#30][opened] sssctl: use systemd D-Bus API

2016-09-21 Thread Pavel Březina
On 09/21/2016 12:31 PM, Jakub Hrozek wrote: On Wed, Sep 21, 2016 at 12:27:16PM +0200, pbrezina wrote: URL: https://github.com/SSSD/sssd/pull/30 Author: pbrezina Title: #30: sssctl: use systemd D-Bus API Action: opened PR body: """ If systemd is used we leverage it's D-Bus API instead of

[SSSD] [sssd PR#29][-Changes requested] tests: Add a regression test for upstream ticket #3131

2016-09-21 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/29 Title: #29: tests: Add a regression test for upstream ticket #3131 Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

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

2016-09-21 Thread celestian
URL: https://github.com/SSSD/sssd/pull/26 Author: celestian Title: #26: KRB5: Fixing FQ name of user in krb5_setup() Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/26/head:pr26 git checkout pr26 From

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

2016-09-21 Thread celestian
URL: https://github.com/SSSD/sssd/pull/26 Title: #26: KRB5: Fixing FQ name of user in krb5_setup() celestian commented: """ So, all notes addressed (fq_name too). I pushed new version. """ See the full comment at https://github.com/SSSD/sssd/pull/26#issuecomment-248580833

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

2016-09-21 Thread celestian
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] [sssd PR#21][comment] IFP: expose user and group unique IDs through DBus

2016-09-21 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus pbrezina commented: """ On 09/21/2016 12:29 PM, Jakub Hrozek wrote: > On Tue, Sep 20, 2016 at 05:24:45AM -0700, sumit-bose wrote: >> > With the SIDs we already have a library thay pretty

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

2016-09-21 Thread celestian
URL: https://github.com/SSSD/sssd/pull/26 Author: celestian Title: #26: KRB5: Fixing FQ name of user in krb5_setup() Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/26/head:pr26 git checkout pr26 From

[SSSD] [sssd PR#30][synchronized] sssctl: use systemd D-Bus API

2016-09-21 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/30 Author: pbrezina Title: #30: sssctl: use systemd D-Bus API Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/30/head:pr30 git checkout pr30 From

[SSSD] Re: [sssd PR#30][opened] sssctl: use systemd D-Bus API

2016-09-21 Thread Jakub Hrozek
On Wed, Sep 21, 2016 at 12:27:16PM +0200, pbrezina wrote: >URL: https://github.com/SSSD/sssd/pull/30 > Author: pbrezina > Title: #30: sssctl: use systemd D-Bus API > Action: opened > > PR body: > """ > If systemd is used we leverage it's D-Bus API instead of running systemctl. > > Resolves:

[SSSD] [sssd PR#30][+Changes requested] sssctl: use systemd D-Bus API

2016-09-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/30 Title: #30: sssctl: use systemd D-Bus API Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

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

2016-09-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus jhrozek commented: """ On Tue, Sep 20, 2016 at 05:24:45AM -0700, sumit-bose wrote: > > With the SIDs we already have a library thay pretty much anyone can call > > and retrieve the SID for

[SSSD] [sssd PR#30][opened] sssctl: use systemd D-Bus API

2016-09-21 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/30 Author: pbrezina Title: #30: sssctl: use systemd D-Bus API Action: opened PR body: """ If systemd is used we leverage it's D-Bus API instead of running systemctl. Resolves: https://fedorahosted.org/sssd/ticket/3056 """ To pull the PR as Git branch:

[SSSD] Re: fedorahosted.org sunset

2016-09-21 Thread Jakub Hrozek
On Fri, Sep 16, 2016 at 12:58:38PM +0200, Lukas Slebodnik wrote: > On (16/09/16 12:52), Jakub Hrozek wrote: > >On Fri, Sep 16, 2016 at 12:34:26PM +0200, Fabiano Fidêncio wrote: > >> Howdy! > >> > >> On Fri, Sep 16, 2016 at 11:09 AM, Jakub Hrozek wrote: > >> > Hi, > >> > > >>

[SSSD] [sssd PR#4][+Changes requested] Added small tweaks to enable SSSD to be compiled with the musl libc

2016-09-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/4 Title: #4: Added small tweaks to enable SSSD to be compiled with the musl libc Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] Re: [PATCH] Fix offline resolution of autofs maps and netgroups

2016-09-21 Thread Lukas Slebodnik
On (21/09/16 11:43), Lukas Slebodnik wrote: >On (19/09/16 12:11), Jakub Hrozek wrote: >>On Mon, Sep 19, 2016 at 10:41:13AM +0200, Lukas Slebodnik wrote: >>> On (19/09/16 10:06), Lukas Slebodnik wrote: >>> >On (12/09/16 00:25), Jakub Hrozek wrote: >>> >>From fd71b7ed7c588cc22402573b9877020234949e22

[SSSD] Re: [PATCH] Fix offline resolution of autofs maps and netgroups

2016-09-21 Thread Lukas Slebodnik
On (19/09/16 12:11), Jakub Hrozek wrote: >On Mon, Sep 19, 2016 at 10:41:13AM +0200, Lukas Slebodnik wrote: >> On (19/09/16 10:06), Lukas Slebodnik wrote: >> >On (12/09/16 00:25), Jakub Hrozek wrote: >> >>From fd71b7ed7c588cc22402573b9877020234949e22 Mon Sep 17 00:00:00 2001 >> >>From: Jakub Hrozek

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

2016-09-21 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] [sssd PR#29][comment] tests: Add a regression test for upstream ticket #3131

2016-09-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/29 Title: #29: tests: Add a regression test for upstream ticket #3131 jhrozek commented: """ On Wed, Sep 21, 2016 at 02:12:42AM -0700, Pavel Březina wrote: > On 09/21/2016 11:02 AM, Jakub Hrozek wrote: > > Assigned #29

[SSSD] [sssd PR#29][synchronized] tests: Add a regression test for upstream ticket #3131

2016-09-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/29 Author: jhrozek Title: #29: tests: Add a regression test for upstream ticket #3131 Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/29/head:pr29 git checkout pr29 From

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

2016-09-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/13 Title: #13: MEMBEROF: Don't resolve members if they are removed jhrozek commented: """ I think all review requests were addressed, the code now looks good to me and just reverting the non-code changes make the tests fail. So I'm adding the Accepted

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

2016-09-21 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#29][+Changes requested] tests: Add a regression test for upstream ticket #3131

2016-09-21 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/29 Title: #29: tests: Add a regression test for upstream ticket #3131 Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#29][comment] tests: Add a regression test for upstream ticket #3131

2016-09-21 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/29 Title: #29: tests: Add a regression test for upstream ticket #3131 pbrezina commented: """ On 09/21/2016 11:02 AM, Jakub Hrozek wrote: > Assigned #29 to @pbrezina > . > > — > You are

[SSSD] [sssd PR#12][comment] failover: proceed normally when no new server is found

2016-09-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/12 Title: #12: failover: proceed normally when no new server is found jhrozek commented: """ @pbrezina I will push this one if you ack https://github.com/SSSD/sssd/pull/29 so that we also have tests.. """ See the full comment at

[SSSD] Re: [PATCH] Remove double semicolon at the end of line

2016-09-21 Thread Pavel Březina
On 09/20/2016 03:36 PM, Lukas Slebodnik wrote: ehlo, simple patch attached + shell based unit test. LS Ack. ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#29][opened] tests: Add a regression test for upstream ticket #3131

2016-09-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/29 Author: jhrozek Title: #29: tests: Add a regression test for upstream ticket #3131 Action: opened PR body: """ Tests that running two duplicate SRV resolution queries succeeds and returns a valid host name. """ To pull the PR as Git branch: git

[SSSD] [sssd PR#12][comment] failover: proceed normally when no new server is found

2016-09-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/12 Title: #12: failover: proceed normally when no new server is found jhrozek commented: """ ACK. I wrote a simple test and will submit a PR separately. """ See the full comment at https://github.com/SSSD/sssd/pull/12#issuecomment-248552171

[SSSD] [sssd PR#12][+Accepted] failover: proceed normally when no new server is found

2016-09-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/12 Title: #12: failover: proceed normally when no new server is found Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to