Re: [SSSD] [PATCH] LDAP: Only convert direct parents' ghost attribute to member

2012-11-21 Thread Jakub Hrozek
On Tue, Nov 20, 2012 at 01:22:52PM +0100, Pavel Březina wrote: On 11/20/2012 02:09 AM, Simo Sorce wrote: On Mon, 2012-11-19 at 23:31 +0100, Jakub Hrozek wrote: On Mon, Nov 19, 2012 at 05:51:53PM +0100, Pavel Březina wrote: On 11/18/2012 11:58 PM, Jakub Hrozek wrote: [PATCH 1/2] SYSDB: Use the

Re: [SSSD] [PATCH] MONITOR: Fix off-by-one error in add_string_to_list

2012-11-21 Thread Jakub Hrozek
On Tue, Nov 20, 2012 at 12:22:29PM -0500, Stephen Gallagher wrote: On Tue 20 Nov 2012 12:01:33 PM EST, Jakub Hrozek wrote: On Tue, Nov 20, 2012 at 07:43:04AM -0500, Stephen Gallagher wrote: On 11/20/2012 05:08 AM, Jakub Hrozek wrote: I noticed a seemigly random crash in the monitor while

Re: [SSSD] Wrong DB version (got 0.13 expected 0.10)

2012-11-21 Thread Jakub Hrozek
On Tue, Nov 20, 2012 at 07:20:43PM -0500, Ariel Barria wrote: sorry for two email, did not know existed this ldb-tools :) Was your issue resolves? What was the problem? ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

Re: [SSSD] [PATCH] idmap: Silence DEBUG messages when dealing with built-in, SIDs.

2012-11-21 Thread Jakub Hrozek
On Tue, Nov 20, 2012 at 03:20:06PM +0100, Pavel Březina wrote: We should propagate the built-in sid error instead of misusing id. Maybe return IDMAP* directly and return errno value in new output parameter. I actually think that using a special ID value is OK. We've been treating the UID and

Re: [SSSD] [PATCHES] sss_userdel and sss_groupdel should invalidate mmap cache

2012-11-21 Thread Ondrej Kos
On 11/20/2012 02:29 PM, Michal Židek wrote: On 11/20/2012 02:22 PM, Michal Židek wrote: Patch 1: sss_cache refactor. See patch description for more details. Patch 2: Remove mmap cache properly in sss_userdel and sss_groupdel Patches are attached. Thanks Michal

Re: [SSSD] [PATCHES] sss_userdel and sss_groupdel should invalidate mmap cache

2012-11-21 Thread Michal Židek
On 11/21/2012 01:06 PM, Ondrej Kos wrote: On 11/20/2012 02:29 PM, Michal Židek wrote: On 11/20/2012 02:22 PM, Michal Židek wrote: Patch 1: sss_cache refactor. See patch description for more details. Patch 2: Remove mmap cache properly in sss_userdel and sss_groupdel Patches are attached.

Re: [SSSD] [PATCH] Handle compiling FQDN regular expression with old pcre gracefully

2012-11-21 Thread Stephen Gallagher
On Tue 20 Nov 2012 05:27:56 PM EST, Jakub Hrozek wrote: Two users have asked for the option to run the 1.9 series on RHEL5 recently. I haven't tested further than a simple log in as an IPA user with this patch on F-17 and RHEL5. I don't think we should be spending much more time on RHEL5

Re: [SSSD] [PATCHES] sss_userdel and sss_groupdel should invalidate mmap cache

2012-11-21 Thread Ondrej Kos
On 11/21/2012 01:25 PM, Michal Židek wrote: On 11/21/2012 01:06 PM, Ondrej Kos wrote: On 11/20/2012 02:29 PM, Michal Židek wrote: On 11/20/2012 02:22 PM, Michal Židek wrote: Patch 1: sss_cache refactor. See patch description for more details. Patch 2: Remove mmap cache properly in sss_userdel

Re: [SSSD] [PATCH] Handle compiling FQDN regular expression with old pcre gracefully

2012-11-21 Thread Jakub Hrozek
On Wed, Nov 21, 2012 at 07:42:41AM -0500, Stephen Gallagher wrote: On Tue 20 Nov 2012 05:27:56 PM EST, Jakub Hrozek wrote: Two users have asked for the option to run the 1.9 series on RHEL5 recently. I haven't tested further than a simple log in as an IPA user with this patch on F-17 and

Re: [SSSD] [PATCHES] sss_userdel and sss_groupdel should invalidate mmap cache

2012-11-21 Thread Simo Sorce
On Tue, 2012-11-20 at 14:29 +0100, Michal Židek wrote: On 11/20/2012 02:22 PM, Michal Židek wrote: Patch 1: sss_cache refactor. See patch description for more details. Patch 2: Remove mmap cache properly in sss_userdel and sss_groupdel It looks a bit excessive to remove the whole cache when

Re: [SSSD] Wrong DB version (got 0.13 expected 0.10)

2012-11-21 Thread Ariel Barria
not resolved. I meant that ldb-tools for ldbsearch and get version Date: Wed, 21 Nov 2012 11:00:10 +0100 From: jhro...@redhat.com To: sssd-devel@lists.fedorahosted.org Subject: Re: [SSSD] Wrong DB version (got 0.13 expected 0.10) On Tue, Nov 20, 2012 at 07:20:43PM -0500, Ariel Barria

Re: [SSSD] [PATCH] idmap: Silence DEBUG messages when dealing with built-in, SIDs.

2012-11-21 Thread Michal Židek
On 11/21/2012 11:04 AM, Jakub Hrozek wrote: On Tue, Nov 20, 2012 at 03:20:06PM +0100, Pavel Březina wrote: We should propagate the built-in sid error instead of misusing id. Maybe return IDMAP* directly and return errno value in new output parameter. I actually think that using a special ID

Re: [SSSD] [PATCHES] sss_userdel and sss_groupdel should invalidate mmap cache

2012-11-21 Thread Michal Židek
On 11/21/2012 02:31 PM, Simo Sorce wrote: On Tue, 2012-11-20 at 14:29 +0100, Michal Židek wrote: On 11/20/2012 02:22 PM, Michal Židek wrote: Patch 1: sss_cache refactor. See patch description for more details. Patch 2: Remove mmap cache properly in sss_userdel and sss_groupdel It looks a bit

Re: [SSSD] [PATCHES] sss_userdel and sss_groupdel should invalidate mmap cache

2012-11-21 Thread Simo Sorce
On Wed, 2012-11-21 at 15:56 +0100, Michal Židek wrote: On 11/21/2012 02:31 PM, Simo Sorce wrote: On Tue, 2012-11-20 at 14:29 +0100, Michal Židek wrote: On 11/20/2012 02:22 PM, Michal Židek wrote: Patch 1: sss_cache refactor. See patch description for more details. Patch 2: Remove mmap

[SSSD] [PATCH] when monitor_quit is call and the process not exists

2012-11-21 Thread Ariel Barria
Hi. well, I'm not sure that this patch have the behavior that is it hope. You have the word. But when monitor_quit is call and the process not exists, this remains in loop and not allow signal. I try it with Monitor Task and nothing. I had to reboot

[SSSD] Fix compare_principal_realm() check broken make check ?

2012-11-21 Thread Simo Sorce
I am getting a make check error: Running suite(s): krb5_utils 93%: Checks: 16, Failures: 1, Errors: 0 ../src/tests/krb5_utils-tests.c:704:F:Helper functions:test_compare_principal_realm:0: Same realm but different_realm is not false. FAIL: krb5-utils-tests It seem related to this commit 2 days

Re: [SSSD] Fix compare_principal_realm() check broken make check ?

2012-11-21 Thread Jakub Hrozek
On Wed, Nov 21, 2012 at 05:24:02PM -0500, Simo Sorce wrote: I am getting a make check error: Running suite(s): krb5_utils 93%: Checks: 16, Failures: 1, Errors: 0 ../src/tests/krb5_utils-tests.c:704:F:Helper functions:test_compare_principal_realm:0: Same realm but different_realm is not