Re: [SSSD] sss_cache flush ssh hosts list.

2014-07-16 Thread Lukas Slebodnik
On (16/07/14 11:46), William wrote: On Tue, 2014-07-15 at 15:57 +0200, Jan Cholasta wrote: On 11.7.2014 03:35, William wrote: Thanks. Could you please rename the option to entry_cache_ssh_host_timeout, so that it's consistent with the rest of the cache timeout options? However, I

[SSSD] sss_client: thread safe initialisation of sss_nss_mc_get_ctx

2014-07-16 Thread Lukas Slebodnik
ehlo, attached patches fix problems with mmap cache in client code. The 1st patch is at least 5th version, because I found few problems in my previous versions myself. I hope you will not find anything else :-) Patches change client code. It will be good to have at least 2 ACKs. How to test?

Re: [SSSD] [PATCH] sss_sifp: set output parameters if attribute is NULL

2014-07-16 Thread Pavel Březina
On 07/10/2014 02:08 PM, Lukas Slebodnik wrote: On (02/07/14 13:09), Pavel Březina wrote: First patch is a minor bug in unit test I found when I was writing new tests. The rest is described in commit message. From de3ed7bf0e9784058241e4b532c72e324e3dd635 Mon Sep 17 00:00:00 2001 From:

Re: [SSSD] [PATCH] sss_sifp: set output parameters if attribute is NULL

2014-07-16 Thread Pavel Březina
On 07/16/2014 01:37 PM, Pavel Březina wrote: On 07/10/2014 02:08 PM, Lukas Slebodnik wrote: On (02/07/14 13:09), Pavel Březina wrote: First patch is a minor bug in unit test I found when I was writing new tests. The rest is described in commit message. From

Re: [SSSD] sss_client: thread safe initialisation of sss_nss_mc_get_ctx

2014-07-16 Thread Sumit Bose
On Wed, Jul 16, 2014 at 11:32:53AM +0200, Lukas Slebodnik wrote: ehlo, attached patches fix problems with mmap cache in client code. The 1st patch is at least 5th version, because I found few problems in my previous versions myself. I hope you will not find anything else :-) Patches

Re: [SSSD] [PATCH] sss_sifp: set output parameters if attribute is NULL

2014-07-16 Thread Lukas Slebodnik
On (16/07/14 13:46), Pavel Březina wrote: On 07/16/2014 01:37 PM, Pavel Březina wrote: On 07/10/2014 02:08 PM, Lukas Slebodnik wrote: On (02/07/14 13:09), Pavel Březina wrote: First patch is a minor bug in unit test I found when I was writing new tests. The rest is described in commit message.

Re: [SSSD] [PATCH] sss_sifp: set output parameters if attribute is NULL

2014-07-16 Thread Jakub Hrozek
On Wed, Jul 16, 2014 at 02:10:16PM +0200, Lukas Slebodnik wrote: Thank you. ACK to all LS Should we push all three at once? Normally we only bump the version number before the release. I suggest we create a blocker ticket for 1.12.1 to remind us to push the last patch instead, in case we

[SSSD] [PATCH] sudo: replace asterisk with escape sequence in host filter

2014-07-16 Thread Pavel Březina
https://fedorahosted.org/sssd/ticket/2377 From 7dec3a954f1d300122723916d9ba21c7677ee313 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrez...@redhat.com Date: Wed, 16 Jul 2014 14:17:24 +0200 Subject: [PATCH] sudo: replace asterisk with escape sequence in host filter Resolves:

Re: [SSSD] [PATCH] sudo: replace asterisk with escape sequence in host filter

2014-07-16 Thread Jakub Hrozek
On Wed, Jul 16, 2014 at 02:49:01PM +0200, Pavel Březina wrote: https://fedorahosted.org/sssd/ticket/2377 On IRC you mentioned some problem with this patch and 389DS, is is still the case? ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

Re: [SSSD] [PATCH] sudo: replace asterisk with escape sequence in host filter

2014-07-16 Thread Pavel Březina
On 07/16/2014 02:59 PM, Jakub Hrozek wrote: On Wed, Jul 16, 2014 at 02:49:01PM +0200, Pavel Březina wrote: https://fedorahosted.org/sssd/ticket/2377 On IRC you mentioned some problem with this patch and 389DS, is is still the case? Nope. ___

[SSSD] [PATCHES] sss_case = preserving

2014-07-16 Thread Michal Židek
Hi, patches for ticket https://fedorahosted.org/sssd/ticket/2367 are in attachment. Michal From 071b3ca8cd9a194c8cb287f9abca2fe7c58323a2 Mon Sep 17 00:00:00 2001 From: Michal Zidek mzi...@redhat.com Date: Tue, 15 Jul 2014 12:00:36 -0400 Subject: [PATCH 1/3] Add function confdb_set_string. ---

Re: [SSSD] [PATCHES] sss_case = preserving

2014-07-16 Thread Michal Židek
On 07/16/2014 03:40 PM, Michal Židek wrote: Hi, patches for ticket https://fedorahosted.org/sssd/ticket/2367 are in attachment. Michal I forgot to add reference to the ticket in the patch description. New patches are attached. Michal From 5f7094e93988d6aa475ea33ceb87d380737b4795 Mon Sep

Re: [SSSD] [PATCH] sss_sifp: set output parameters if attribute is NULL

2014-07-16 Thread Lukas Slebodnik
On (16/07/14 14:38), Jakub Hrozek wrote: On Wed, Jul 16, 2014 at 02:10:16PM +0200, Lukas Slebodnik wrote: Thank you. ACK to all LS Should we push all three at once? Normally we only bump the version number before the release. I suggest we create a blocker ticket for 1.12.1 to remind us to

Re: [SSSD] sss_client: thread safe initialisation of sss_nss_mc_get_ctx

2014-07-16 Thread Lukas Slebodnik
On (16/07/14 14:02), Sumit Bose wrote: On Wed, Jul 16, 2014 at 11:32:53AM +0200, Lukas Slebodnik wrote: ehlo, attached patches fix problems with mmap cache in client code. The 1st patch is at least 5th version, because I found few problems in my previous versions myself. I hope you will not

Re: [SSSD] [PATCH] Use sss_strerror instead of strerror

2014-07-16 Thread Lukas Slebodnik
On (16/07/14 16:37), Michal Židek wrote: On 05/11/2014 11:10 PM, Jakub Hrozek wrote: I haven't tested this patch to be honest, but then again, there's not so much to test except compiling and a bit of sanity testing.. I prefer Michal's version of using sss_strerror explicitly instead of the route

[SSSD] [PATCH] IPA: incorrect expansion of group membership when encountering a non-POSIX group

2014-07-16 Thread Pavel Reichl
Hello, please see attached patches. Regards, Pavel Reichl From b0bb8006c024046ae3aca2f5837489cd12fe2cd7 Mon Sep 17 00:00:00 2001 From: Pavel Reichl prei...@redhat.com Date: Wed, 16 Jul 2014 13:33:58 +0100 Subject: [PATCH 1/2] IPA: new attribute map for non-posix groups Create new set of

Re: [SSSD] [PATCH] Retry system bus connection once messagebus is up

2014-07-16 Thread Jakub Hrozek
On Tue, Jul 15, 2014 at 02:47:21PM +0200, Pavel Březina wrote: On 07/14/2014 11:54 AM, Jakub Hrozek wrote: On Tue, Jul 08, 2014 at 08:42:43PM +0200, Jakub Hrozek wrote: The 1.11 version will be sent out separately. Here are the 1.11 patches. There are no functional differences, just some

Re: [SSSD] [PATCH] sss_sifp: set output parameters if attribute is NULL

2014-07-16 Thread Jakub Hrozek
On Wed, Jul 16, 2014 at 03:49:37PM +0200, Lukas Slebodnik wrote: On (16/07/14 14:38), Jakub Hrozek wrote: On Wed, Jul 16, 2014 at 02:10:16PM +0200, Lukas Slebodnik wrote: Thank you. ACK to all LS Should we push all three at once? Normally we only bump the version number before

Re: [SSSD] [PATCH] sss_sifp: set output parameters if attribute is NULL

2014-07-16 Thread Jakub Hrozek
On Wed, Jul 16, 2014 at 05:46:07PM +0200, Jakub Hrozek wrote: I will only push the first two patches master: 973be642f3d33ba21ea9c06791295f09efcdba46 b7080f1a2c6c97224c41f6347ca3743e1054faec ___ sssd-devel mailing list

Re: [SSSD] sss_cache flush ssh hosts list.

2014-07-16 Thread William
Doesn't appear to be related to anything I have changed I don't think ... You forgot to change usage of sysdb_store_ssh_host in sysdb_ssh-tests. tests cannot be compiled. (make check) CC src/tests/sysdb_ssh_tests-sysdb_ssh-tests.o src/tests/sysdb_ssh-tests.c:179:43: error:

Re: [SSSD] [PATCH] LDAP: tokengroups do not work with id_provider=ldap

2014-07-16 Thread Jakub Hrozek
On 10 Jul 2014, at 16:38, Pavel Reichl prei...@redhat.com wrote: Hello, please see attached patches. I found out that if we go with approach introduced in previous version (in case of LDAP provider assume SID comes from default domain) this can lead to resolutions of SIDs like