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

2016-09-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/25 Title: #25: TESTS: Add integration tests for the proxy provider of sssd-secrets jhrozek commented: """ On Tue, Sep 20, 2016 at 08:40:08AM -0700, lslebodn wrote: > On (20/09/16 05:02), Jakub Hrozek wrote: > >On Tue, Sep 20, 2016 at 04:45:53AM -0700,

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

2016-09-20 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/25 Title: #25: TESTS: Add integration tests for the proxy provider of sssd-secrets lslebodn commented: """ On (20/09/16 05:02), Jakub Hrozek wrote: >On Tue, Sep 20, 2016 at 04:45:53AM -0700, lslebodn wrote: >> On (20/09/16 04:09), Jakub Hrozek wrote: >> >I

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

2016-09-20 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/19/2016 02:26 PM, tequeter wrote: > Yes, I considered creating a |group_attributes| instead. However it > would have taken more refactoring to avoid duplicating

[SSSD] Re: [PATCH] sss_override: improve --debug description

2016-09-20 Thread Pavel Březina
On 09/16/2016 02:44 PM, Lukas Slebodnik wrote: On (31/08/16 09:25), Justin Stephenson wrote: On 08/31/2016 06:59 AM, Pavel Březina wrote: On 08/30/2016 08:52 PM, Justin Stephenson wrote: On 08/05/2016 11:45 AM, Lukas Slebodnik wrote: On (15/03/16 12:40), Pavel Březina wrote: On 12/09/2015

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

2016-09-20 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/21 Title: #21: IFP: expose user and group unique IDs through DBus sumit-bose commented: """ > With the SIDs we already have a library thay pretty much anyone can call and > retrieve the SID for ID. But not for GUIDs.. CC @sbose-rh for another > opinion..

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

2016-09-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/25 Title: #25: TESTS: Add integration tests for the proxy provider of sssd-secrets jhrozek commented: """ On Mon, Sep 19, 2016 at 06:54:12AM -0700, lslebodn wrote: > lslebodn commented on this pull request. > > > > > +os.chmod(config.CONF_PATH,

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

2016-09-20 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 03:11:59AM -0700, Pavel Březina wrote: > On 09/19/2016 02:26 PM, tequeter wrote: > > Yes, I considered creating a |group_attributes| instead.

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

2016-09-20 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/25 Title: #25: TESTS: Add integration tests for the proxy provider of sssd-secrets lslebodn commented: """ On (20/09/16 04:09), Jakub Hrozek wrote: >I was getting random failures where the socket was not yet ready without >the sleep, like this: >

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

2016-09-20 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-20 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#25][comment] TESTS: Add integration tests for the proxy provider of sssd-secrets

2016-09-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/25 Title: #25: TESTS: Add integration tests for the proxy provider of sssd-secrets jhrozek commented: """ On Tue, Sep 20, 2016 at 04:45:53AM -0700, lslebodn wrote: > On (20/09/16 04:09), Jakub Hrozek wrote: > >I was getting random failures where the socket

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

2016-09-20 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] [PATCH] Remove double semicolon at the end of line

2016-09-20 Thread Lukas Slebodnik
ehlo, simple patch attached + shell based unit test. LS From e21ab30e79bbf033f6fab8549b6f7e79e4f4d194 Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Sat, 17 Sep 2016 21:05:36 +0200 Subject: [PATCH 1/4] Remove double semicolon at the end of line ---

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

2016-09-20 Thread celestian
URL: https://github.com/SSSD/sssd/pull/26 Title: #26: KRB5: Fixing FQ name of user in krb5_setup() celestian commented: """ New version is pushed. """ See the full comment at https://github.com/SSSD/sssd/pull/26#issuecomment-248295259 ___ sssd-devel

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

2016-09-20 Thread celestian
URL: https://github.com/SSSD/sssd/pull/25 Title: #25: TESTS: Add integration tests for the proxy provider of sssd-secrets celestian commented: """ So, code LGTM, thanks for Lukas's watchful eye. I pushed patch to CI. """ See the full comment at

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

2016-09-20 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