Re: [Freeipa-devel] [PATCH 0047] Avoid manual connection management outside ldap_query()

2012-08-28 Thread Petr Spacek
On 08/22/2012 03:35 PM, Adam Tkac wrote: On Mon, Aug 13, 2012 at 03:15:52PM +0200, Petr Spacek wrote: Hello, this patch improves connection management in bind-dyndb-ldap and closes https://fedorahosted.org/bind-dyndb-ldap/ticket/68 . It should prevent all deadlocks on connection pool in

Re: [Freeipa-devel] [PATCH 0047] Avoid manual connection management outside ldap_query()

2012-08-28 Thread Petr Spacek
On 08/28/2012 09:57 AM, Adam Tkac wrote: On Tue, Aug 28, 2012 at 08:51:31AM +0200, Petr Spacek wrote: On 08/22/2012 03:35 PM, Adam Tkac wrote: On Mon, Aug 13, 2012 at 03:15:52PM +0200, Petr Spacek wrote: Hello, this patch improves connection management in bind-dyndb-ldap and closes

Re: [Freeipa-devel] [PATCH 0049] Fix two memory leaks in persistent search

2012-08-28 Thread Petr Spacek
On 08/22/2012 03:43 PM, Adam Tkac wrote: On Tue, Aug 14, 2012 at 02:32:55PM +0200, Petr Spacek wrote: Hello, This patch fixes two memory leaks in persistent search. Ack. From 892f1d5c59a97cdad7a2807ecd172488605ab181 Mon Sep 17 00:00:00 2001 From: Petr Spacek pspa...@redhat.com Date: Tue,

[Freeipa-devel] [PATCH 0009] Improves deletion of PTR records in ipa host-del.

2012-08-28 Thread Tomas Babej
Hi, Command ipa host-del with --updatedns now can deal both with hosts which zones are in FQDN form with or without a trailing dot. https://fedorahosted.org/freeipa/ticket/2809 Tomas ___ Freeipa-devel mailing list Freeipa-devel@redhat.com

Re: [Freeipa-devel] Paging in Web UI

2012-08-28 Thread John Dennis
On 08/28/2012 09:30 AM, Petr Vobornik wrote: I would like to point out a problem in Web UI related to paging and suggest a possible solution: Current implementation of paging in Web UI is not serving it purpose which should be to do operations faster and be able to find all users. How current

Re: [Freeipa-devel] [PATCH] 197 Fixed search in HBAC test

2012-08-28 Thread Endi Sukma Dewata
On 8/21/2012 9:03 AM, Petr Vobornik wrote: Search in HBAC test wasn't working because expired flag wasn't set. https://fedorahosted.org/freeipa/ticket/2931 Notes: HBAC facets don't have refresh button. They can be refreshed by changing filter and searching. If one search with same filter, it

Re: [Freeipa-devel] [PATCH] 198 Revert change causing failure in test automation

2012-08-28 Thread Endi Sukma Dewata
On 8/22/2012 2:50 AM, Petr Vobornik wrote: Move of click handler in patch for #2834 causes failure of automation tests. This patch reverts the problematic part. It should not affect function of fix for #2834. https://fedorahosted.org/freeipa/ticket/3014 ACK. -- Endi S. Dewata

Re: [Freeipa-devel] [PATCH] 200 Fix issue which broke setup of Web UI unit tests

2012-08-28 Thread Endi Sukma Dewata
On 8/27/2012 5:52 AM, Petr Vobornik wrote: Fix issue which broke setup of Web UI unit tests Web UI itself wasn't negatively affected. Issue introduced in be144da672e0634f7aaeff69d662cbc4d11aff0f (#2897). https://fedorahosted.org/freeipa/ticket/2897 ACK. -- Endi S. Dewata

Re: [Freeipa-devel] [PATCH] 201 Successful action notification

2012-08-28 Thread Endi Sukma Dewata
On 8/27/2012 5:57 AM, Petr Vobornik wrote: User was not notified about success of actions executed from action list, action panel or facet control bar. This patch adds IPA.notify_success(message) call. It creates a yellow notification area with supplied message in Web UI header in the middle of

Re: [Freeipa-devel] [PATCH] 202 Password policy paging with proper sorting

2012-08-28 Thread Endi Sukma Dewata
ACK. Some comments below. On 8/27/2012 10:51 AM, Petr Vobornik wrote: This patch adds option to disable sorting when paging. It allowed to enable paging in password policy with order of items untouched (they are sorted on server side by priority). Is the sorting we see in the UI and CLI

[Freeipa-devel] [PATCH] 297 Update Contributors.txt file

2012-08-28 Thread Martin Kosek
Update list of active developers working on IPA. --- I also tried to add people from community contributing on our wiki pages on freeipa.org. ACKed by Rob off-list. Pushed to master, ipa-3-0. Martin From 1d72a448a3cff60e7dab3ec25ee73583bd52d0fa Mon Sep 17 00:00:00 2001 From: Martin Kosek

Re: [Freeipa-devel] Paging in Web UI

2012-08-28 Thread Petr Vobornik
On 08/28/2012 04:17 PM, John Dennis wrote: On 08/28/2012 09:30 AM, Petr Vobornik wrote: I would like to point out a problem in Web UI related to paging and suggest a possible solution: Current implementation of paging in Web UI is not serving it purpose which should be to do operations faster

Re: [Freeipa-devel] Paging in Web UI

2012-08-28 Thread Endi Sukma Dewata
On 8/28/2012 8:30 AM, Petr Vobornik wrote: I would like to point out a problem in Web UI related to paging and suggest a possible solution: Current implementation of paging in Web UI is not serving it purpose which should be to do operations faster and be able to find all users. How current

Re: [Freeipa-devel] Paging in Web UI

2012-08-28 Thread Endi Sukma Dewata
On 8/28/2012 11:23 AM, Petr Vobornik wrote: Your possible solution does not address how many results are fetched (unless I misunderstood). If paging is enabled it doesn't, but it expects, that admin will disable it for larger setups.For smaller setups it isn't of much an issue. If paging is

[Freeipa-devel] [PATCH] 1049 validate MLS value

2012-08-28 Thread Rob Crittenden
Validate that the MLS value in a SELinux user map user is in the range c0..c1023. Existing tests validate correct values, empty values, I'm just adding a high value test. rob From 94cbfc3aeaed521319a9da371a44f3873a46657a Mon Sep 17 00:00:00 2001 From: Rob Crittenden rcrit...@redhat.com Date:

Re: [Freeipa-devel] [PATCH] Patch to allow IPA to work with dogtag 10 on f18

2012-08-28 Thread Ade Lee
I have not opened a certmonger bug, but here is a patch to fix the relevant code in certmonger. Nalin, please review and commit. I tested by renewing one of the dogtag system certs (the ocsp signing cert) Ade On Mon, 2012-08-27 at 09:40 -0400, Rob Crittenden wrote: Petr Viktorin wrote: On