[SSSD] Re: [SSSD} [PATCH] Remove no longer used code

2016-08-29 Thread Petr Cech
On 08/26/2016 04:59 PM, Jakub Hrozek wrote: On Mon, Aug 15, 2016 at 02:58:50PM +0200, Fabiano Fidêncio wrote: Those 3 patches are from Jakub and I've just done some minor adjustments and add myself as co-author of the first 2 patches. CI has passed:

[SSSD] Re: [RFCv3] NSS tlog integration

2016-08-29 Thread Simo Sorce
On Mon, 2016-08-29 at 22:47 +0200, Sumit Bose wrote: > About Simo's concern. If there are groups defined in the > session_recording configuration you look up those groups with > cache_req_group_by_name_send(). As long as the groups are in the cache > and not expired they are returned directly from

[SSSD] Re: [RFCv3] NSS tlog integration

2016-08-29 Thread Sumit Bose
On Thu, Aug 25, 2016 at 01:03:19PM -0400, Simo Sorce wrote: > On Thu, 2016-08-25 at 14:24 +0300, Nikolai Kondrashov wrote: > > Hi Simo, > > > > Thanks for looking at the patches and for the feedback! > > I'm replying below. > > > > On 08/24/2016 11:24 PM, Simo Sorce wrote: > > > On Tue,

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Lukas Slebodnik
On (29/08/16 18:11), Fabiano Fidêncio wrote: >On Mon, Aug 29, 2016 at 6:03 PM, Lukas Slebodnik wrote: >> On (29/08/16 16:38), Lukas Slebodnik wrote: >>>On (29/08/16 16:05), Fabiano Fidêncio wrote: On Mon, Aug 29, 2016 at 3:51 PM, Lukas Slebodnik

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Fabiano Fidêncio
On Mon, Aug 29, 2016 at 6:03 PM, Lukas Slebodnik wrote: > On (29/08/16 16:38), Lukas Slebodnik wrote: >>On (29/08/16 16:05), Fabiano Fidêncio wrote: >>>On Mon, Aug 29, 2016 at 3:51 PM, Lukas Slebodnik wrote: On (29/08/16 14:47), Lukas Slebodnik

[SSSD] Re: [PATCH] BUILD: Remove leftover after sysdb refactoring

2016-08-29 Thread Lukas Slebodnik
On (29/08/16 18:00), Fabiano Fidêncio wrote: >On Mon, Aug 29, 2016 at 5:52 PM, Lukas Slebodnik wrote: >> ehlo, >> >> simple patch is attached. >> >> LS >> > >Acked-by: Fabiano Fidêncio > master: * 4229ffb929bd7029f8b94d92099032d3350f5cf4 LS

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Lukas Slebodnik
On (29/08/16 16:38), Lukas Slebodnik wrote: >On (29/08/16 16:05), Fabiano Fidêncio wrote: >>On Mon, Aug 29, 2016 at 3:51 PM, Lukas Slebodnik wrote: >>> On (29/08/16 14:47), Lukas Slebodnik wrote: On (29/08/16 14:36), Fabiano Fidêncio wrote: >On Mon, Aug 29, 2016 at

[SSSD] Re: [PATCH] BUILD: Remove leftover after sysdb refactoring

2016-08-29 Thread Fabiano Fidêncio
On Mon, Aug 29, 2016 at 5:52 PM, Lukas Slebodnik wrote: > ehlo, > > simple patch is attached. > > LS > > ___ > sssd-devel mailing list > sssd-devel@lists.fedorahosted.org >

[SSSD] Re: [PATCHSET] BUILD: Improve a little bit the "prerelease-*" targets

2016-08-29 Thread Lukas Slebodnik
On (29/08/16 14:42), Fabiano Fidêncio wrote: >On Mon, Aug 29, 2016 at 2:30 PM, Lukas Slebodnik wrote: >> On (29/08/16 14:22), Fabiano Fidêncio wrote: >>>On Mon, Aug 29, 2016 at 1:39 PM, Lukas Slebodnik wrote: On (17/08/16 22:14), Fabiano Fidêncio

[SSSD] Re: [PATCH] sudo man page: say that we support IPA schema

2016-08-29 Thread Justin Stephenson
On 08/10/2016 04:33 PM, Dan Lavu wrote: I asked Lukas this but he wasn't positive, is the objectClasses different when adding 'ldap_sudo_search_base' ? Or is it just location? Eitherway, I think this is going to be a little more concise, "When SSSD is configured and using the IPA provider,

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Jakub Hrozek
On Mon, Aug 29, 2016 at 03:57:47PM +0200, Fabiano Fidêncio wrote: > On Mon, Aug 29, 2016 at 3:53 PM, Jakub Hrozek wrote: > > On Mon, Aug 29, 2016 at 03:51:06PM +0200, Lukas Slebodnik wrote: > >> On (29/08/16 14:47), Lukas Slebodnik wrote: > >> >On (29/08/16 14:36), Fabiano

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Lukas Slebodnik
On (29/08/16 16:05), Fabiano Fidêncio wrote: >On Mon, Aug 29, 2016 at 3:51 PM, Lukas Slebodnik wrote: >> On (29/08/16 14:47), Lukas Slebodnik wrote: >>>On (29/08/16 14:36), Fabiano Fidêncio wrote: On Mon, Aug 29, 2016 at 2:27 PM, Lukas Slebodnik

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Fabiano Fidêncio
On Mon, Aug 29, 2016 at 3:51 PM, Lukas Slebodnik wrote: > On (29/08/16 14:47), Lukas Slebodnik wrote: >>On (29/08/16 14:36), Fabiano Fidêncio wrote: >>>On Mon, Aug 29, 2016 at 2:27 PM, Lukas Slebodnik wrote: On (29/08/16 14:15), Fabiano Fidêncio

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Fabiano Fidêncio
On Mon, Aug 29, 2016 at 3:53 PM, Jakub Hrozek wrote: > On Mon, Aug 29, 2016 at 03:51:06PM +0200, Lukas Slebodnik wrote: >> On (29/08/16 14:47), Lukas Slebodnik wrote: >> >On (29/08/16 14:36), Fabiano Fidêncio wrote: >> >>On Mon, Aug 29, 2016 at 2:27 PM, Lukas Slebodnik

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Jakub Hrozek
On Mon, Aug 29, 2016 at 03:51:06PM +0200, Lukas Slebodnik wrote: > On (29/08/16 14:47), Lukas Slebodnik wrote: > >On (29/08/16 14:36), Fabiano Fidêncio wrote: > >>On Mon, Aug 29, 2016 at 2:27 PM, Lukas Slebodnik > >>wrote: > >>> On (29/08/16 14:15), Fabiano Fidêncio wrote: >

[SSSD] Re: 'no primary group ID provided' when using AD in ldap mode

2016-08-29 Thread Jakub Hrozek
On Mon, Aug 29, 2016 at 04:49:57AM -, Daniel Hermans wrote: > HI, > sorry I didn't get any response on sssd-users This was just because nobody had the time to answer until today (most developers are in EU..). It's still better to post on the users list because the archives will be searchable

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Lukas Slebodnik
On (29/08/16 14:36), Fabiano Fidêncio wrote: >On Mon, Aug 29, 2016 at 2:27 PM, Lukas Slebodnik wrote: >> On (29/08/16 14:15), Fabiano Fidêncio wrote: >>>On Mon, Aug 29, 2016 at 12:36 PM, Lukas Slebodnik >>>wrote: On (29/08/16 12:08), Fabiano

[SSSD] Re: [PATCHSET] BUILD: Improve a little bit the "prerelease-*" targets

2016-08-29 Thread Fabiano Fidêncio
On Mon, Aug 29, 2016 at 2:30 PM, Lukas Slebodnik wrote: > On (29/08/16 14:22), Fabiano Fidêncio wrote: >>On Mon, Aug 29, 2016 at 1:39 PM, Lukas Slebodnik wrote: >>> On (17/08/16 22:14), Fabiano Fidêncio wrote: Howdy! I've been using a lot

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Fabiano Fidêncio
On Mon, Aug 29, 2016 at 2:27 PM, Lukas Slebodnik wrote: > On (29/08/16 14:15), Fabiano Fidêncio wrote: >>On Mon, Aug 29, 2016 at 12:36 PM, Lukas Slebodnik wrote: >>> On (29/08/16 12:08), Fabiano Fidêncio wrote: On Mon, Aug 29, 2016 at 11:12 AM, Jakub

[SSSD] Re: [PATCHSET] BUILD: Improve a little bit the "prerelease-*" targets

2016-08-29 Thread Lukas Slebodnik
On (29/08/16 14:22), Fabiano Fidêncio wrote: >On Mon, Aug 29, 2016 at 1:39 PM, Lukas Slebodnik wrote: >> On (17/08/16 22:14), Fabiano Fidêncio wrote: >>>Howdy! >>> >>>I've been using a lot the "make prerelease-rpms" command and I've >>>found out the fact that all the

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Lukas Slebodnik
On (29/08/16 14:15), Fabiano Fidêncio wrote: >On Mon, Aug 29, 2016 at 12:36 PM, Lukas Slebodnik wrote: >> On (29/08/16 12:08), Fabiano Fidêncio wrote: >>>On Mon, Aug 29, 2016 at 11:12 AM, Jakub Hrozek wrote: On Mon, Aug 29, 2016 at 10:38:46AM +0200,

[SSSD] Re: [PATCHSET] BUILD: Improve a little bit the "prerelease-*" targets

2016-08-29 Thread Fabiano Fidêncio
On Mon, Aug 29, 2016 at 1:39 PM, Lukas Slebodnik wrote: > On (17/08/16 22:14), Fabiano Fidêncio wrote: >>Howdy! >> >>I've been using a lot the "make prerelease-rpms" command and I've >>found out the fact that all the resulting rpms of 2 (or more builds) >>are put in the same

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Fabiano Fidêncio
On Mon, Aug 29, 2016 at 12:36 PM, Lukas Slebodnik wrote: > On (29/08/16 12:08), Fabiano Fidêncio wrote: >>On Mon, Aug 29, 2016 at 11:12 AM, Jakub Hrozek wrote: >>> On Mon, Aug 29, 2016 at 10:38:46AM +0200, Lukas Slebodnik wrote: On (29/08/16 07:09),

[SSSD] Re: [PATCH] Create pidfile after responders have started

2016-08-29 Thread Lukas Slebodnik
On (26/08/16 13:16), Victor Tapia wrote: >Hi, > >Sorry for the late response, I've been out for nearly two weeks. I'm >attaching a new version of the patch to review. > >> Does upstart/init script need to be updated as well? >> We have some generic in src/sysv/sssd.in > >Upstart does need to keep

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Lukas Slebodnik
On (29/08/16 12:08), Fabiano Fidêncio wrote: >On Mon, Aug 29, 2016 at 11:12 AM, Jakub Hrozek wrote: >> On Mon, Aug 29, 2016 at 10:38:46AM +0200, Lukas Slebodnik wrote: >>> On (29/08/16 07:09), Fabiano Fidêncio wrote: >>> >Hoiwdy! >>> > >>> > >>> >On Fri, Aug 19, 2016 at 1:08

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Fabiano Fidêncio
On Mon, Aug 29, 2016 at 11:12 AM, Jakub Hrozek wrote: > On Mon, Aug 29, 2016 at 10:38:46AM +0200, Lukas Slebodnik wrote: >> On (29/08/16 07:09), Fabiano Fidêncio wrote: >> >Hoiwdy! >> > >> > >> >On Fri, Aug 19, 2016 at 1:08 AM, Fabiano Fidêncio >>

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Jakub Hrozek
On Mon, Aug 29, 2016 at 10:38:46AM +0200, Lukas Slebodnik wrote: > On (29/08/16 07:09), Fabiano Fidêncio wrote: > >Hoiwdy! > > > > > >On Fri, Aug 19, 2016 at 1:08 AM, Fabiano Fidêncio > >wrote: > >> This patch is a first attempt to make "make intgcheck" less > >>

[SSSD] Re: [PATCH] BUILD: Break the intgcheck target into 5 smaller targets: intgcheck-{prepare,configure,build,run,clean}

2016-08-29 Thread Lukas Slebodnik
On (29/08/16 07:09), Fabiano Fidêncio wrote: >Hoiwdy! > > >On Fri, Aug 19, 2016 at 1:08 AM, Fabiano Fidêncio wrote: >> This patch is a first attempt to make "make intgcheck" less >> painful/time consuming than it is now. >> >> Although the patch provides a good improvement on

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

2016-08-29 Thread Jakub Hrozek
On Sun, Aug 28, 2016 at 03:31:05PM -0400, Justin Stephenson wrote: > > > Hello, > > > > > > I wanted to provide some feedback on the Contribute wiki page from an > > > outsiders perspective after spending time writing a few patches for sssd. > > > I > > > hope it will be of some value to improve

[SSSD] Re: [PATCH] SYSDB: Fix uninitialized scalar variable

2016-08-29 Thread Lukas Slebodnik
On (29/08/16 09:25), Fabiano Fidêncio wrote: >On Mon, Aug 29, 2016 at 9:23 AM, Lukas Slebodnik wrote: >> ehlo, >> >> simple patch is attached. >> >> LS >> > >Hmm. It passed during my re-work in the code, sorry :-\ > >Acked-by: Fabiano Fidêncio master: *

[SSSD] Re: [PATCH] SYSDB: Fix uninitialized scalar variable

2016-08-29 Thread Fabiano Fidêncio
On Mon, Aug 29, 2016 at 9:23 AM, Lukas Slebodnik wrote: > ehlo, > > simple patch is attached. > > LS > > ___ > sssd-devel mailing list > sssd-devel@lists.fedorahosted.org >

[SSSD] [PATCH] SYSDB: Fix uninitialized scalar variable

2016-08-29 Thread Lukas Slebodnik
ehlo, simple patch is attached. LS >From dbb1a0ee3e60f96630055b6a28de74eea1096f3b Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Mon, 29 Aug 2016 09:13:49 +0200 Subject: [PATCH] SYSDB: Fix uninitialized scalar variable The boolean variable newly_created could be used