[SSSD] Re: [RFCv3] NSS tlog integration

2016-08-24 Thread Simo Sorce
On Tue, 2016-08-23 at 17:24 +0300, Nikolai Kondrashov wrote: > Hi everyone, > > Attached is the third version of work-in-progress SSSD/tlog > integration > patches. I'm sending them in the hope that somebody takes a look and > perhaps > points out some wrong bits I can fix before I'm too

[SSSD] Re: tasks for newcomers or non-developers

2016-08-24 Thread Justin Stephenson
On 08/04/2016 04:20 AM, Jakub Hrozek wrote: Hi, Over the last couple of weeks, I've talked to several people, mostly engineers with not too much development experience, who said they would like to start contributing little fixes to SSSD. I would like to find some tasks that they can easily

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

2016-08-24 Thread Petr Cech
Hello, I am fighting with adding new option to sssd.conf. I slowly running out of breath. I know proxy could be id, auth or chpass provider. I don't know where is the right place for my option. And the second issue is it breaks test for SSSD config. :-( Is there anyone who would like to join

[SSSD] Re: [PATCH] LDAP: Fixing of removing netgroup from cache

2016-08-24 Thread Lukas Slebodnik
On (24/08/16 13:46), Lukas Slebodnik wrote: >On (24/08/16 10:53), Petr Cech wrote: >>On 08/24/2016 09:15 AM, Lukas Slebodnik wrote: >>> On (22/08/16 16:39), Petr Cech wrote: >>> > On 08/19/2016 05:18 PM, Lukas Slebodnik wrote: >>> > > Thank you for test but I would appraciate a little bit simpler

[SSSD] Re: [PATCH] sssd_netgroup.py: Resolve nested netgroups

2016-08-24 Thread Lukas Slebodnik
On (24/08/16 13:51), Petr Cech wrote: >On 08/24/2016 09:33 AM, Petr Cech wrote: >> Hi Lukas, >> >> I didn't run CI tests because your new code is not used yet. >> But I run my tests for nested_group with your updated version. >> Everything works how we expected. >> Code LGTM. >> >> => ACK > >CI

[SSSD] Re: [PATCH] sssd_netgroup.py: Resolve nested netgroups

2016-08-24 Thread Petr Cech
On 08/24/2016 09:33 AM, Petr Cech wrote: Hi Lukas, I didn't run CI tests because your new code is not used yet. But I run my tests for nested_group with your updated version. Everything works how we expected. Code LGTM. => ACK CI passed with my patches:

[SSSD] Re: [PATCH] LDAP: Fixing of removing netgroup from cache

2016-08-24 Thread Lukas Slebodnik
On (24/08/16 10:53), Petr Cech wrote: >On 08/24/2016 09:15 AM, Lukas Slebodnik wrote: >> On (22/08/16 16:39), Petr Cech wrote: >> > On 08/19/2016 05:18 PM, Lukas Slebodnik wrote: >> > > Thank you for test but I would appraciate a little bit simpler solution. >> > >

[SSSD] Re: PROXY: Use the fqname when converting to lowercase

2016-08-24 Thread Lukas Slebodnik
On (24/08/16 11:26), Jakub Hrozek wrote: >On Wed, Aug 24, 2016 at 10:48:21AM +0200, Fabiano Fidêncio wrote: >> On Wed, Aug 24, 2016 at 10:43 AM, Lukas Slebodnik >> wrote: >> > On (24/08/16 10:37), Fabiano Fidêncio wrote: >> >>On Wed, Aug 24, 2016 at 10:33 AM, Lukas Slebodnik

[SSSD] Re: [PATCH] watchdog: cope with time shift

2016-08-24 Thread Fabiano Fidêncio
On Wed, Aug 24, 2016 at 11:55 AM, Pavel Březina wrote: > On 08/24/2016 11:41 AM, Fabiano Fidêncio wrote: >> >> On Wed, 2016-08-24 at 11:29 +0200, Pavel Březina wrote: >> >>> From 09e5ad3448694d9609bd2a9e89e8a05ef82f66bc Mon Sep 17 00:00:00 >> >> 2001 >>> >>> From:

[SSSD] Re: [PATCH] watchdog: cope with time shift

2016-08-24 Thread Pavel Březina
On 08/24/2016 11:41 AM, Fabiano Fidêncio wrote: On Wed, 2016-08-24 at 11:29 +0200, Pavel Březina wrote: From 09e5ad3448694d9609bd2a9e89e8a05ef82f66bc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= Date: Mon, 22 Aug 2016 13:15:04 +0200 Subject: [PATCH]

[SSSD] Re: [PATCH] watchdog: cope with time shift

2016-08-24 Thread Fabiano Fidêncio
On Wed, 2016-08-24 at 11:29 +0200, Pavel Březina wrote: > From 09e5ad3448694d9609bd2a9e89e8a05ef82f66bc Mon Sep 17 00:00:00 2001 > From: =?UTF-8?q?Pavel=20B=C5=99ezina?= > Date: Mon, 22 Aug 2016 13:15:04 +0200 > Subject: [PATCH] watchdog: cope with time shift >  > Resolves:

[SSSD] [PATCH] watchdog: cope with time shift

2016-08-24 Thread Pavel Březina
scratch build was tested and fix confirmed. I think we should also reschedule all existing tevent timers, we don't have an infrastructure to do it though. From 09e5ad3448694d9609bd2a9e89e8a05ef82f66bc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= Date:

[SSSD] Re: PROXY: Use the fqname when converting to lowercase

2016-08-24 Thread Jakub Hrozek
On Wed, Aug 24, 2016 at 10:48:21AM +0200, Fabiano Fidêncio wrote: > On Wed, Aug 24, 2016 at 10:43 AM, Lukas Slebodnik wrote: > > On (24/08/16 10:37), Fabiano Fidêncio wrote: > >>On Wed, Aug 24, 2016 at 10:33 AM, Lukas Slebodnik > >>wrote: > >>> On

[SSSD] Re: [PATCH] LDAP: Fixing of removing netgroup from cache

2016-08-24 Thread Petr Cech
On 08/24/2016 09:15 AM, Lukas Slebodnik wrote: On (22/08/16 16:39), Petr Cech wrote: On 08/19/2016 05:18 PM, Lukas Slebodnik wrote: Thank you for test but I would appraciate a little bit simpler solution. "memberNisNetgroup"(SYSDB_NETGROUP_MEMBER) Your patch will append

[SSSD] Re: PROXY: Use the fqname when converting to lowercase

2016-08-24 Thread Fabiano Fidêncio
On Wed, Aug 24, 2016 at 10:43 AM, Lukas Slebodnik wrote: > On (24/08/16 10:37), Fabiano Fidêncio wrote: >>On Wed, Aug 24, 2016 at 10:33 AM, Lukas Slebodnik wrote: >>> On (24/08/16 00:59), Fabiano Fidêncio wrote: When saving the user there is a

[SSSD] Re: PROXY: Use the fqname when converting to lowercase

2016-08-24 Thread Lukas Slebodnik
On (24/08/16 10:37), Fabiano Fidêncio wrote: >On Wed, Aug 24, 2016 at 10:33 AM, Lukas Slebodnik wrote: >> On (24/08/16 00:59), Fabiano Fidêncio wrote: >>>When saving the user there is a comparison between the "cased alias" >>>and the "lowercase password name". However, the

[SSSD] Re: PROXY: Use the fqname when converting to lowercase

2016-08-24 Thread Fabiano Fidêncio
On Wed, Aug 24, 2016 at 10:33 AM, Lukas Slebodnik wrote: > On (24/08/16 00:59), Fabiano Fidêncio wrote: >>When saving the user there is a comparison between the "cased alias" >>and the "lowercase password name". However, the first doesn't use fully >>qualified name while the

[SSSD] Re: PROXY: Use the fqname when converting to lowercase

2016-08-24 Thread Lukas Slebodnik
On (24/08/16 00:59), Fabiano Fidêncio wrote: >When saving the user there is a comparison between the "cased alias" >and the "lowercase password name". However, the first doesn't use fully >qualified name while the second does, resulting in a not expected >override of the "nameAlias" attribute of a

[SSSD] Re: [PATCH] sssd_netgroup.py: Resolve nested netgroups

2016-08-24 Thread Petr Cech
On 08/24/2016 07:30 AM, Lukas Slebodnik wrote: On (19/08/16 12:35), Petr Cech wrote: On 08/19/2016 11:42 AM, Petr Cech wrote: On 08/19/2016 11:24 AM, Lukas Slebodnik wrote: On (19/08/16 08:55), Petr Cech wrote: On 08/18/2016 12:22 PM, Petr Cech wrote: On 08/18/2016 12:19 PM, Lukas Slebodnik

[SSSD] Re: [PATCH] LDAP: Fixing of removing netgroup from cache

2016-08-24 Thread Lukas Slebodnik
On (22/08/16 16:39), Petr Cech wrote: >On 08/19/2016 05:18 PM, Lukas Slebodnik wrote: >> Thank you for test but I would appraciate a little bit simpler solution. >> "memberNisNetgroup"(SYSDB_NETGROUP_MEMBER) >> Your patch will append "memberNisNetgroup"(SYSDB_NETGROUP_MEMBER) to missing >>