Re: [Freeipa-devel] [PATCH] 23 Allow one letter net/hostgroups names

2012-05-07 Thread Martin Kosek
On Fri, 2012-05-04 at 10:37 +0200, Ondrej Hamada wrote: On 05/03/2012 05:44 PM, Martin Kosek wrote: On Thu, 2012-05-03 at 17:08 +0200, Ondrej Hamada wrote: On 05/02/2012 05:49 PM, Ondrej Hamada wrote: https://fedorahosted.org/freeipa/ticket/2671 Changed regex validating net/hostgroup

Re: [Freeipa-devel] 389-ds-base-1.2.11.3 has been pushed to testing

2012-05-07 Thread Martin Kosek
On Sat, 2012-05-05 at 08:46 -0600, Rich Megginson wrote: This is the F-17 candidate. It fixes the deadlock and the managed entry deletion issues found by Rob. Please give it some karma before the F-17 deadline on Monday. Thanks! In my VM, this version causes FreeIPA installation error.

Re: [Freeipa-devel] [PATCH 0018] Deadlock detection logic

2012-05-07 Thread Petr Spacek
On 05/03/2012 02:18 PM, Adam Tkac wrote: On Tue, Apr 24, 2012 at 03:52:00PM +0200, Petr Spacek wrote: On 04/24/2012 03:21 PM, Petr Spacek wrote: Hello, this patch adds deadlock detection (based on simple timeout) to current code. If (probable) deadlock is detected, current action is stopped

Re: [Freeipa-devel] [PATCH 0018] Deadlock detection logic

2012-05-07 Thread Adam Tkac
On 05/07/2012 12:35 PM, Petr Spacek wrote: On 05/03/2012 02:18 PM, Adam Tkac wrote: On Tue, Apr 24, 2012 at 03:52:00PM +0200, Petr Spacek wrote: On 04/24/2012 03:21 PM, Petr Spacek wrote: Hello, this patch adds deadlock detection (based on simple timeout) to current code. If (probable)

Re: [Freeipa-devel] [PATCH 0018] Deadlock detection logic

2012-05-07 Thread Petr Spacek
On 05/07/2012 12:50 PM, Adam Tkac wrote: On 05/07/2012 12:35 PM, Petr Spacek wrote: On 05/03/2012 02:18 PM, Adam Tkac wrote: On Tue, Apr 24, 2012 at 03:52:00PM +0200, Petr Spacek wrote: On 04/24/2012 03:21 PM, Petr Spacek wrote: Hello, this patch adds deadlock detection (based on simple

[Freeipa-devel] [PATCH] 78 Redo boolean value encoding

2012-05-07 Thread Jan Cholasta
Hi, this patch changes the way boolean values are encoded to LDAP boolean syntax. The code for encoding boolean values is moved from the Parameter class to the Encoder class, where the rest of LDAP encoding takes place. The patch removes encoding code from the Parameter class altogether, as

[Freeipa-devel] [PATCH 0020] Separate LDAP result from LDAP connection, fix deadlock.

2012-05-07 Thread Petr Spacek
Hello, this patch fixes https://fedorahosted.org/bind-dyndb-ldap/ticket/66: Plugin deadlocks during new zone load when connections == 1. It fixes structural problem, when LDAP query result was tied with LDAP connection up. It wasn't possible to release connection and work with query result

Re: [Freeipa-devel] [PATCHES] 0042-43 Fix internal server errors with empty options

2012-05-07 Thread Martin Kosek
On Mon, 2012-04-30 at 14:17 +0200, Petr Viktorin wrote: On 04/27/2012 03:23 PM, Petr Viktorin wrote: Empty values in reverse_member options, and attattr/setattr/delattr, caused internal server errors. We convert empty values to None and bypass normal validation, so they need special

Re: [Freeipa-devel] DNS zone serial number updates [#2554]

2012-05-07 Thread Petr Spacek
Hello, I found following reply in unsent mail, so I resending it to complete original thread. It's just for record, because original idea was obsoleted by recent discussion on Thursday meeting. Petr^2 Spacek On 04/18/2012 05:53 PM, Simo Sorce wrote: On Wed, 2012-04-18 at 17:21 +0200,

Re: [Freeipa-devel] [PATCH] 78 Redo boolean value encoding

2012-05-07 Thread Martin Kosek
On Mon, 2012-05-07 at 14:48 +0200, Jan Cholasta wrote: Hi, this patch changes the way boolean values are encoded to LDAP boolean syntax. The code for encoding boolean values is moved from the Parameter class to the Encoder class, where the rest of LDAP encoding takes place. The patch

Re: [Freeipa-devel] [PATCH] 78 Redo boolean value encoding

2012-05-07 Thread Jan Cholasta
On 7.5.2012 17:59, Martin Kosek wrote: On Mon, 2012-05-07 at 14:48 +0200, Jan Cholasta wrote: Hi, this patch changes the way boolean values are encoded to LDAP boolean syntax. The code for encoding boolean values is moved from the Parameter class to the Encoder class, where the rest of LDAP

Re: [Freeipa-devel] DNS zone serial number updates [#2554]: local SOA approach

2012-05-07 Thread Petr Spacek
Hello, on the last meeting there was another approach to $SUBJ$ discussed: Each DNS server will maintain its own serial number value independently from other servers. Pros: Should be simpler to implement; no DS plugin required. Cons: Slave DNS servers cannot fall-back to other masters,

Re: [Freeipa-devel] [PATCHES] 124-132 Inconsistent ways to show/change entry status

2012-05-07 Thread Endi Sukma Dewata
The code works, it's ACKed. I have some comments below: On 5/2/2012 8:33 AM, Petr Vobornik wrote: This bunch of patches are implementing ticket #2247. They introduce some new logic and types of internal objects. There might be design issues (mainly in state evaluation). I would appreciate some

Re: [Freeipa-devel] [PATCHES] 124-132 Inconsistent ways to show/change entry status

2012-05-07 Thread Endi Sukma Dewata
On 5/3/2012 8:26 AM, Petr Vobornik wrote: On 05/03/2012 03:19 PM, Petr Vobornik wrote: I found that limitation of maximum pkey length in facet header is not working well. Attaching patch #134 which actually calculates it. I found useless line in the patch. Corrected version attached. Try