Re: [SSSD] [PATCH] CI: Do not try to find memory leaks with valgrind

2014-09-16 Thread Nikolai Kondrashov
On 09/16/2014 10:53 PM, Lukas Slebodnik wrote: From 4b20f833f71c9014b0874b5a899d71f0bdbf9ca3 Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Tue, 2 Sep 2014 12:00:03 +0200 Subject: [PATCH] CI: Do not try to find memory leaks with valgrind We use talloc and valgrind needn't find memory leak

Re: [SSSD] [PATCH] CI: Do not try to find memory leaks with valgrind

2014-09-16 Thread Lukas Slebodnik
On (16/09/14 17:00), Simo Sorce wrote: >On Tue, 16 Sep 2014 21:53:56 +0200 >Lukas Slebodnik wrote: > >> On (02/09/14 15:38), Lukas Slebodnik wrote: >> >On (02/09/14 14:23), Nikolai Kondrashov wrote: >> >>On 09/02/2014 02:13 PM, Lukas Slebodnik wrote: >> >>>On (02/09/14 13:27), Nikolai Kondrashov w

Re: [SSSD] [PATCH] CI: Do not try to find memory leaks with valgrind

2014-09-16 Thread Simo Sorce
On Tue, 16 Sep 2014 21:53:56 +0200 Lukas Slebodnik wrote: > On (02/09/14 15:38), Lukas Slebodnik wrote: > >On (02/09/14 14:23), Nikolai Kondrashov wrote: > >>On 09/02/2014 02:13 PM, Lukas Slebodnik wrote: > >>>On (02/09/14 13:27), Nikolai Kondrashov wrote: > On 09/02/2014 01:02 PM, Lukas Sleb

Re: [SSSD] [PATCH] CI: Do not try to find memory leaks with valgrind

2014-09-16 Thread Lukas Slebodnik
On (02/09/14 15:38), Lukas Slebodnik wrote: >On (02/09/14 14:23), Nikolai Kondrashov wrote: >>On 09/02/2014 02:13 PM, Lukas Slebodnik wrote: >>>On (02/09/14 13:27), Nikolai Kondrashov wrote: On 09/02/2014 01:02 PM, Lukas Slebodnik wrote: >We use talloc and valgrind needn't find memory leaks

[SSSD] [PATCH] sysdb: sysdb_search_group_by_name should work like sysdb_search_user_by_name

2014-09-16 Thread Sumit Bose
Hi, I came across this when trying to resolve the SID of a group with the group's name. bye, Sumit From c26f19e3d3cca24e45ed36e395693862f5e1664f Mon Sep 17 00:00:00 2001 From: Sumit Bose Date: Fri, 12 Sep 2014 17:40:11 +0200 Subject: [PATCH] sysdb: sysdb_search_group_by_name should work like sy

Re: [SSSD] [PATCH] CI: Add missing debian dependency

2014-09-16 Thread Jakub Hrozek
On Thu, Sep 11, 2014 at 08:01:18PM +0200, Michal Židek wrote: > On 09/08/2014 10:15 AM, Nikolai Kondrashov wrote: > >On 09/08/2014 10:42 AM, Lukas Slebodnik wrote: > >>Fix for one problem is attached. (missing dependency: make) > > > >I don't have a freshly-installed Debian handy to verify this pro

Re: [SSSD] [PATCH] CI: Preserve mock config timestamps

2014-09-16 Thread Jakub Hrozek
On Thu, Sep 11, 2014 at 08:07:09PM +0200, Michal Židek wrote: > On 09/03/2014 10:36 AM, Nikolai Kondrashov wrote: > >Hi everyone, > > > >This patch adds preserving of mock configuration timestamps, so mock root > >cache is not rebuilt unnecessarily, shaving off about a minute of CI run > >time. > >

Re: [SSSD] [PATCHES] TESTS: Fix memory leaks

2014-09-16 Thread Jakub Hrozek
On Tue, Sep 16, 2014 at 02:25:11PM +0200, Jakub Hrozek wrote: > On Fri, Sep 05, 2014 at 03:39:24PM +0300, Nikolai Kondrashov wrote: > > Hi everyone, > > > > These patches fix a few test memory leaks detected by Valgrind. > > > > While these are not important for test functionality, fixing them wi

Re: [SSSD] [PATCHES] TESTS: Fix memory leaks

2014-09-16 Thread Jakub Hrozek
On Fri, Sep 05, 2014 at 03:39:24PM +0300, Nikolai Kondrashov wrote: > Hi everyone, > > These patches fix a few test memory leaks detected by Valgrind. > > While these are not important for test functionality, fixing them will let us > keep Valgrind reports clean and enable binary Valgrind test re

Re: [SSSD] [PATCH] LDAP: Do not clobber return value when multiple controls are returned

2014-09-16 Thread Jakub Hrozek
On Mon, Sep 15, 2014 at 03:33:14PM +0200, Pavel Reichl wrote: > > On 09/15/2014 03:02 PM, Jakub Hrozek wrote: > >Hi, > > > >please see the attached patch and its commit message. An alternative > >might be to use a different variable for the result (as we did in 1.9) > >before the new error codes.

Re: [SSSD] [PATCH] CI: Improve usability of ci script

2014-09-16 Thread Martin Kosek
On 09/15/2014 08:04 PM, Nikolai Kondrashov wrote: > On 09/12/2014 06:18 PM, Lukas Slebodnik wrote: >> On (12/09/14 16:43), Nikolai Kondrashov wrote: >>> On 09/12/2014 10:22 AM, Lukas Slebodnik wrote: On (11/09/14 23:09), Nikolai Kondrashov wrote: > On 09/11/2014 09:16 PM, Lukas Slebodnik w

Re: [SSSD] [PATCH] name2sid: Check negative cache for users and groups

2014-09-16 Thread Jakub Hrozek
On Tue, Sep 16, 2014 at 01:00:12PM +0200, Sumit Bose wrote: > On Tue, Sep 16, 2014 at 12:03:33PM +0200, Jakub Hrozek wrote: > > On Fri, Sep 05, 2014 at 05:54:59PM +0200, Sumit Bose wrote: > > > Hi, > > > > > > I found this while testing other stuff. > > > > > > bye, > > > Sumit > > > > The patch

Re: [SSSD] [PATCH] name2sid: Check negative cache for users and groups

2014-09-16 Thread Sumit Bose
On Tue, Sep 16, 2014 at 12:03:33PM +0200, Jakub Hrozek wrote: > On Fri, Sep 05, 2014 at 05:54:59PM +0200, Sumit Bose wrote: > > Hi, > > > > I found this while testing other stuff. > > > > bye, > > Sumit > > The patch works as expected, so ACK. I will just fix one DEBUG message > before pushing i

Re: [SSSD] [PATCH] name2sid: Check negative cache for users and groups

2014-09-16 Thread Jakub Hrozek
On Fri, Sep 05, 2014 at 05:54:59PM +0200, Sumit Bose wrote: > Hi, > > I found this while testing other stuff. > > bye, > Sumit The patch works as expected, so ACK. I will just fix one DEBUG message before pushing if you don't mind, see inline. Also, while testing I realized that when looking up

Re: [SSSD] SSSD 1.11.7?

2014-09-16 Thread Jakub Hrozek
On Tue, Sep 16, 2014 at 11:10:38AM +0200, Lukas Slebodnik wrote: > On (16/09/14 10:24), Jakub Hrozek wrote: > >On Mon, Sep 15, 2014 at 06:42:45PM +0200, Jakub Hrozek wrote: > >> Hi, > >> > >> I think we have all the patches for 1.11.7 in. Can we tag and release > >> it? > > > >Unless there's a 'no

Re: [SSSD] SSSD 1.11.7?

2014-09-16 Thread Lukas Slebodnik
On (16/09/14 10:24), Jakub Hrozek wrote: >On Mon, Sep 15, 2014 at 06:42:45PM +0200, Jakub Hrozek wrote: >> Hi, >> >> I think we have all the patches for 1.11.7 in. Can we tag and release >> it? > >Unless there's a 'no' vote I'll tag and release 1.11.7 tomorrow, Wed >17th. It seems taht ticket #238

[SSSD] [PATCH] MAN: AD is allowed value of subdomains_provider

2014-09-16 Thread Jakub Hrozek
Reported by Scott Poore. >From 22c5680aa75078418c9e4f99794b96e476bb4800 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Tue, 16 Sep 2014 11:07:29 +0200 Subject: [PATCH] MAN: AD is allowed value of subdomains_provider https://fedorahosted.org/sssd/ticket/2442 --- src/man/sssd.conf.5.xml | 9 +++

Re: [SSSD] SSSD 1.11.7?

2014-09-16 Thread Jakub Hrozek
On Mon, Sep 15, 2014 at 06:42:45PM +0200, Jakub Hrozek wrote: > Hi, > > I think we have all the patches for 1.11.7 in. Can we tag and release > it? Unless there's a 'no' vote I'll tag and release 1.11.7 tomorrow, Wed 17th. ___ sssd-devel mailing list ss

Re: [SSSD] [PATCH] CI: Do not try to find memory leaks with valgrind

2014-09-16 Thread Nikolai Kondrashov
On 09/15/2014 10:57 PM, Lukas Slebodnik wrote: On (02/09/14 15:38), Lukas Slebodnik wrote: On (02/09/14 14:23), Nikolai Kondrashov wrote: On 09/02/2014 02:13 PM, Lukas Slebodnik wrote: On (02/09/14 13:27), Nikolai Kondrashov wrote: On 09/02/2014 01:02 PM, Lukas Slebodnik wrote: We use talloc