[SSSD] Re: MONITOR: Add disable_netlink sssd.conf option

2016-09-05 Thread Lukas Slebodnik
On (05/09/16 15:54), Petr Cech wrote: > > >On 09/05/2016 09:45 AM, Lukas Slebodnik wrote: >> On (02/09/16 15:34), Petr Cech wrote: >> > >> > >> > On 09/02/2016 03:31 PM, Justin Stephenson wrote: >> > > On 09/02/2016 05:23 AM, Petr Cech wrote: >> > > > On 09/01/2016 03:36 PM, Justin Stephenson

[SSSD] Re: [PATCH] PROXY: Adding proxy_max_children option

2016-09-05 Thread Petr Cech
On 09/05/2016 04:05 PM, Fabiano Fidêncio wrote: Petr, On Mon, Sep 5, 2016 at 3:43 PM, Petr Cech wrote: On 09/05/2016 09:57 AM, Fabiano Fidêncio wrote: Petr, I see you have updated the OPT_MAX_CHILDREN_DEFAULT to 10 instead of 50. However, you haven't update the value on

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-09-05 Thread Jakub Hrozek
On Mon, Sep 05, 2016 at 03:32:48PM +0200, Lukas Slebodnik wrote: > On (05/09/16 15:24), Jakub Hrozek wrote: > >On Mon, Sep 05, 2016 at 02:31:31PM +0200, Fabiano Fidêncio wrote: > >> On Mon, Sep 5, 2016 at 11:59 AM, Fabiano Fidêncio > >> wrote: > >> > Petr, > >> > > >> > I

[SSSD] Re: [PATCH] PROXY: Adding proxy_max_children option

2016-09-05 Thread Fabiano Fidêncio
Petr, On Mon, Sep 5, 2016 at 3:43 PM, Petr Cech wrote: > On 09/05/2016 09:57 AM, Fabiano Fidêncio wrote: >> >> Petr, >> >> I see you have updated the OPT_MAX_CHILDREN_DEFAULT to 10 instead of >> 50. However, you haven't update the value on sssd.conf.5.xml: >> >> diff --git

[SSSD] Re: MONITOR: Add disable_netlink sssd.conf option

2016-09-05 Thread Petr Cech
On 09/05/2016 09:45 AM, Lukas Slebodnik wrote: On (02/09/16 15:34), Petr Cech wrote: On 09/02/2016 03:31 PM, Justin Stephenson wrote: On 09/02/2016 05:23 AM, Petr Cech wrote: On 09/01/2016 03:36 PM, Justin Stephenson wrote: On 08/30/2016 03:54 AM, Jakub Hrozek wrote: On Sat, Aug 27,

[SSSD] Re: [PATCH] PROXY: Adding proxy_max_children option

2016-09-05 Thread Petr Cech
On 09/05/2016 09:57 AM, Fabiano Fidêncio wrote: Petr, I see you have updated the OPT_MAX_CHILDREN_DEFAULT to 10 instead of 50. However, you haven't update the value on sssd.conf.5.xml: diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml index

[SSSD] Re: [RFC] Use GNULIB's compiler warning code

2016-09-05 Thread Fabiano Fidêncio
On Thu, Aug 11, 2016 at 2:33 PM, Fabiano Fidêncio wrote: > Howdy! > > I've suggested, a long time ago, that we could start making use of > GNULIB's compiler warnings from 'manywarnings' module. This is > basically what we have been doing in a few projects that I (used to >

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-09-05 Thread Petr Cech
On 09/05/2016 03:32 PM, Lukas Slebodnik wrote: On (05/09/16 15:24), Jakub Hrozek wrote: On Mon, Sep 05, 2016 at 02:31:31PM +0200, Fabiano Fidêncio wrote: On Mon, Sep 5, 2016 at 11:59 AM, Fabiano Fidêncio wrote: Petr, I went through your patches and in general they look

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-09-05 Thread Lukas Slebodnik
On (05/09/16 15:24), Jakub Hrozek wrote: >On Mon, Sep 05, 2016 at 02:31:31PM +0200, Fabiano Fidêncio wrote: >> On Mon, Sep 5, 2016 at 11:59 AM, Fabiano Fidêncio >> wrote: >> > Petr, >> > >> > I went through your patches and in general they look good to me. >> > However, I

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-09-05 Thread Petr Cech
On 09/05/2016 03:24 PM, Jakub Hrozek wrote: On Mon, Sep 05, 2016 at 02:31:31PM +0200, Fabiano Fidêncio wrote: On Mon, Sep 5, 2016 at 11:59 AM, Fabiano Fidêncio wrote: Petr, I went through your patches and in general they look good to me. However, I haven't done any tests

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-09-05 Thread Jakub Hrozek
On Mon, Sep 05, 2016 at 02:31:31PM +0200, Fabiano Fidêncio wrote: > On Mon, Sep 5, 2016 at 11:59 AM, Fabiano Fidêncio wrote: > > Petr, > > > > I went through your patches and in general they look good to me. > > However, I haven't done any tests yet with your patches (and

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-09-05 Thread Fabiano Fidêncio
On Mon, Sep 5, 2016 at 11:59 AM, Fabiano Fidêncio wrote: > Petr, > > I went through your patches and in general they look good to me. > However, I haven't done any tests yet with your patches (and I'll do > it after lunch). I've done some tests and I've been able to see the

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-09-05 Thread Petr Cech
On 09/05/2016 11:59 AM, Fabiano Fidêncio wrote: Petr, I went through your patches and in general they look good to me. However, I haven't done any tests yet with your patches (and I'll do it after lunch). Please, below you can see a few comments. Feel completely free to ignore the first one if

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-09-05 Thread Fabiano Fidêncio
Petr, I went through your patches and in general they look good to me. However, I haven't done any tests yet with your patches (and I'll do it after lunch). Please, below you can see a few comments. Feel completely free to ignore the first one if you feel like doing it, it's just a minor :-) For

[SSSD] Re: [PATCH] PROXY: Adding proxy_max_children option

2016-09-05 Thread Fabiano Fidêncio
Petr, I see you have updated the OPT_MAX_CHILDREN_DEFAULT to 10 instead of 50. However, you haven't update the value on sssd.conf.5.xml: diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml index ae291e0fc8f2f9afabcdf32f18a5ec12252f..1bf3e799047d9c722487be8657bbee5cfd479cdd 100644

[SSSD] Re: MONITOR: Add disable_netlink sssd.conf option

2016-09-05 Thread Lukas Slebodnik
On (02/09/16 15:34), Petr Cech wrote: > > >On 09/02/2016 03:31 PM, Justin Stephenson wrote: >> On 09/02/2016 05:23 AM, Petr Cech wrote: >> > On 09/01/2016 03:36 PM, Justin Stephenson wrote: >> > > On 08/30/2016 03:54 AM, Jakub Hrozek wrote: >> > > > On Sat, Aug 27, 2016 at 12:54:53PM -0400, Justin

[SSSD] Re: [PATCH] PROXY: Adding proxy_max_children option

2016-09-05 Thread Petr Cech
Bump. -- Petr^4 Čech ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-09-05 Thread Petr Cech
Bump. -- Petr^4 Čech ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org