Re: [SSSD] [PATCH] MAN: Clarify that changing ID mapping options might require purging the cache

2014-02-25 Thread Dmitri Pal
On 02/25/2014 02:39 PM, Jakub Hrozek wrote: On Tue, Feb 25, 2014 at 11:58:41AM -0500, Dmitri Pal wrote: On 02/25/2014 11:11 AM, Jakub Hrozek wrote: Hi, the attached patch addresses #2252. I tried to make it clear that removing the cache should only be done while online, but I'm open to any fur

Re: [SSSD] [Patch] Cmocka unit test for negcache.c module

2014-02-25 Thread Jakub Hrozek
On Tue, Feb 25, 2014 at 10:00:00PM +0100, Lukas Slebodnik wrote: > On (25/02/14 16:40), Jakub Hrozek wrote: > >On Mon, Feb 17, 2014 at 04:47:09PM +0100, Lukas Slebodnik wrote: > >> The test(s) don't pass because they cannot pass. > >> You used function create_dom_test_ctx and param with filter_grou

Re: [SSSD] [Patch] Cmocka unit test for negcache.c module

2014-02-25 Thread Lukas Slebodnik
On (25/02/14 16:40), Jakub Hrozek wrote: >On Mon, Feb 17, 2014 at 04:47:09PM +0100, Lukas Slebodnik wrote: >> The test(s) don't pass because they cannot pass. >> You used function create_dom_test_ctx and param with filter_groups and >> filter_users and result of this is configuration: >> >> [domai

Re: [SSSD] [PATCH] LDAP: Setup periodic task only once.

2014-02-25 Thread Lukas Slebodnik
On (25/02/14 21:05), Jakub Hrozek wrote: >On Tue, Feb 25, 2014 at 08:05:45PM +0100, Lukas Slebodnik wrote: >> updated patch is attached. >> >> LS > >I think you sent the original patch by accident. Once again. LS >From d0ca74fd94fd581940b684e3434b571ca327bd5f Mon Sep 17 00:00:00 2001 From: Lukas

Re: [SSSD] [Patch] Cmocka unit test for negcache.c module

2014-02-25 Thread Pallavi Jha
Hi, Updated patch is attached. All tests are passing. Please see the attachment and let me know if any further changes are to be made. Thanks! Pallavi On 25 February 2014 21:25, Jakub Hrozek wrote: > On Mon, Feb 17, 2014 at 04:47:09PM +0100, Lukas Slebodnik wrote: > > The test(s) don't pass

Re: [SSSD] [INI] Refactoring UTF8 conversion

2014-02-25 Thread Lukas Slebodnik
On (21/02/14 18:31), Dmitri Pal wrote: >Hello, > >Refactoring for ticket: https://fedorahosted.org/sssd/ticket/2192 > >-- >Thank you, >Dmitri Pal > >Sr. Engineering Manager for IdM portfolio >Red Hat Inc. > > >--- >Looking to carve out IT costs? >www.redhat.com/carveout

Re: [SSSD] [PATCH] LDAP: Setup periodic task only once.

2014-02-25 Thread Jakub Hrozek
On Tue, Feb 25, 2014 at 08:05:45PM +0100, Lukas Slebodnik wrote: > updated patch is attached. > > LS I think you sent the original patch by accident. ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/li

Re: [SSSD] [PATCH] UTIL: Sanitize spaces.

2014-02-25 Thread Jakub Hrozek
On Tue, Feb 25, 2014 at 07:07:28PM +0100, Lukas Slebodnik wrote: > On (25/02/14 14:23), Jakub Hrozek wrote: > >On Mon, Feb 24, 2014 at 11:48:23AM +0100, Lukas Slebodnik wrote: > >> ehlo, > >> > >> resendig patch from trac to review. > >> > >> Original patches submitted by: mpesari(Thanks!!) > >>

Re: [SSSD] [PATCH] MAN: Clarify that changing ID mapping options might require purging the cache

2014-02-25 Thread Jakub Hrozek
On Tue, Feb 25, 2014 at 08:39:26PM +0100, Jakub Hrozek wrote: > On Tue, Feb 25, 2014 at 11:58:41AM -0500, Dmitri Pal wrote: > > On 02/25/2014 11:11 AM, Jakub Hrozek wrote: > > >Hi, > > > > > >the attached patch addresses #2252. I tried to make it clear that > > >removing the cache should only be do

Re: [SSSD] [PATCH] MAN: Clarify that changing ID mapping options might require purging the cache

2014-02-25 Thread Jakub Hrozek
On Tue, Feb 25, 2014 at 11:58:41AM -0500, Dmitri Pal wrote: > On 02/25/2014 11:11 AM, Jakub Hrozek wrote: > >Hi, > > > >the attached patch addresses #2252. I tried to make it clear that > >removing the cache should only be done while online, but I'm open to any > >further suggestions. > > > > > >__

Re: [SSSD] [PATCH] LDAP: Setup periodic task only once.

2014-02-25 Thread Lukas Slebodnik
On (24/02/14 20:45), Jakub Hrozek wrote: >On Mon, Feb 24, 2014 at 08:34:35PM +0100, Jakub Hrozek wrote: >> On Sat, Feb 22, 2014 at 12:18:01AM +0100, Lukas Slebodnik wrote: >> > ehlo, >> > >> > If id provider is {ipa, ad} periodic task will be stared in >> > sssm_{ipa,ad}_init >> > If you enable e

Re: [SSSD] [PATCH] UTIL: Sanitize spaces.

2014-02-25 Thread Lukas Slebodnik
On (25/02/14 14:23), Jakub Hrozek wrote: >On Mon, Feb 24, 2014 at 11:48:23AM +0100, Lukas Slebodnik wrote: >> ehlo, >> >> resendig patch from trac to review. >> >> Original patches submitted by: mpesari(Thanks!!) >> >> It can cause problems if user will hit spaces before entering username. >> (e

Re: [SSSD] [PATCH][ding_libs] Add version symbol files

2014-02-25 Thread Lukas Slebodnik
On (21/02/14 11:21), Lukas Slebodnik wrote: >On (21/02/14 10:54), Lukas Slebodnik wrote: >>On (16/01/14 10:17), Lukas Slebodnik wrote: >>>On (13/01/14 19:42), Lukas Slebodnik wrote: ehlo, attached patch addresses ticket #2193 I have just a question about refarray/libref_array.

Re: [SSSD] [PATCH] MAN: Clarify that changing ID mapping options might require purging the cache

2014-02-25 Thread Dmitri Pal
On 02/25/2014 11:11 AM, Jakub Hrozek wrote: Hi, the attached patch addresses #2252. I tried to make it clear that removing the cache should only be done while online, but I'm open to any further suggestions. ___ sssd-devel mailing list sssd-devel@lis

Re: [SSSD] [PATCH] IPA: check ranges for collisions before saving them

2014-02-25 Thread Jakub Hrozek
On Tue, Feb 25, 2014 at 05:15:18PM +0100, Sumit Bose wrote: > On Mon, Feb 24, 2014 at 07:12:00PM +0100, Jakub Hrozek wrote: > > On Thu, Feb 20, 2014 at 10:34:44AM +0100, Sumit Bose wrote: > > > On Wed, Feb 19, 2014 at 04:23:55PM +0100, Jakub Hrozek wrote: > > > > On Wed, Feb 12, 2014 at 03:43:51PM

[SSSD] [PATCH] MAN: Clarify that changing ID mapping options might require purging the cache

2014-02-25 Thread Jakub Hrozek
Hi, the attached patch addresses #2252. I tried to make it clear that removing the cache should only be done while online, but I'm open to any further suggestions. >From a914d6b060670a86bfbb8657295c221d51cf02a6 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Tue, 25 Feb 2014 17:09:00 +0100 Subj

Re: [SSSD] [Patch] Cmocka unit test for negcache.c module

2014-02-25 Thread Jakub Hrozek
On Mon, Feb 17, 2014 at 04:47:09PM +0100, Lukas Slebodnik wrote: > The test(s) don't pass because they cannot pass. > You used function create_dom_test_ctx and param with filter_groups and > filter_users and result of this is configuration: > > [domain/nss_test] > filter_users = testuser1 > filter

Re: [SSSD] Cmocka-unit-test-for-functions-getpwuid-added

2014-02-25 Thread Jakub Hrozek
On Sat, Feb 15, 2014 at 01:04:36AM +0545, Pallavi Jha wrote: > Hi, > > Attached is a patch with the additional test functions for getpwuid* > functions. I think I did not get the flow of the data in program properly. > That is why getting similar errors. > > Please have a look into the patch and

Re: [SSSD] [PATCH] UTIL: Sanitize spaces.

2014-02-25 Thread Jakub Hrozek
On Mon, Feb 24, 2014 at 11:48:23AM +0100, Lukas Slebodnik wrote: > ehlo, > > resendig patch from trac to review. > > Original patches submitted by: mpesari(Thanks!!) > > It can cause problems if user will hit spaces before entering username. > (e.g in gdm). Spaces are ignored by LDAP; it's bette

Re: [SSSD] [PATCH][PUSHED] Fix DEBUG message formatting

2014-02-25 Thread Stephen Gallagher
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/25/2014 07:41 AM, Jakub Hrozek wrote: > Hi, > > I pushed the attached patch to master under the one-liner rule. > It's just amending the multi-line DEBUG messages so that there is a > space between the two words. > > Posthumous ack -BEGIN

[SSSD] [PATCH][PUSHED] Fix DEBUG message formatting

2014-02-25 Thread Jakub Hrozek
Hi, I pushed the attached patch to master under the one-liner rule. It's just amending the multi-line DEBUG messages so that there is a space between the two words. >From 4d69d511ee4843cbdacde2f73ed378182ead83f6 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Tue, 25 Feb 2014 13:38:44 +0100 Sub

[SSSD] Design Discussion: Simple D-Bus API

2014-02-25 Thread Pavel Březina
On 02/24/2014 01:17 PM, Jakub Hrozek wrote: On Mon, Feb 24, 2014 at 12:32:04PM +0100, Pavel Březina wrote: https://fedorahosted.org/sssd/wiki/DesignDocs/DBusResponder Hi, I couldn't find the original thread so I'm starting a new one. The design was reviewed in a private thread mostly as the A