[Freeipa-devel] [PATCH] 531 Increase stack size for Web UI builder

2013-12-13 Thread Petr Vobornik
Web UI build fails on some architectures or configuration due to StackOverflow. This patch increases the stack size to solve it. 512k is usually enough but we encountered fail on ppc64 even with 2m, therefore the 8m. The build is single threaded so it shouldn't waste much memory. -- Petr

Re: [Freeipa-devel] [PATCH] 531 Increase stack size for Web UI builder

2013-12-13 Thread Alexander Bokovoy
On Fri, 13 Dec 2013, Petr Vobornik wrote: Web UI build fails on some architectures or configuration due to StackOverflow. This patch increases the stack size to solve it. 512k is usually enough but we encountered fail on ppc64 even with 2m, therefore the 8m. The build is single threaded so it

[Freeipa-devel] ou, st, l missing from organizationalPerson (Was: FreeIPA 3.3.latest failing tests: config_mod)

2013-12-13 Thread Petr Viktorin
I finally got to investigating this failure. It seems ou, along with a bunch of other attributes, is missing from organizationalPerson in Fedora 20. I don't think it's IPA's fault, as we don't define organizationalPerson. Nathan, could it be related to the new schema parser? f19 has:

Re: [Freeipa-devel] [PATCHES] 0322-0327 New permissions system

2013-12-13 Thread Martin Kosek
On 12/12/2013 05:17 PM, Petr Viktorin wrote: On 12/12/2013 02:00 PM, Martin Kosek wrote: On 12/06/2013 11:49 AM, Petr Viktorin wrote: On 12/02/2013 01:04 PM, Martin Kosek wrote: On 12/01/2013 11:46 PM, Petr Viktorin wrote: This seems to work now. Please tell me what I missed. Design:

Re: [Freeipa-devel] [PATCH] 531 Increase stack size for Web UI builder

2013-12-13 Thread Petr Vobornik
On 12/13/2013 10:28 AM, Alexander Bokovoy wrote: On Fri, 13 Dec 2013, Petr Vobornik wrote: Web UI build fails on some architectures or configuration due to StackOverflow. This patch increases the stack size to solve it. 512k is usually enough but we encountered fail on ppc64 even with 2m,

Re: [Freeipa-devel] [PATCH] 531 Increase stack size for Web UI builder

2013-12-13 Thread Alexander Bokovoy
On Fri, 13 Dec 2013, Petr Vobornik wrote: Web UI build fails on some architectures or configuration due to StackOverflow. This patch increases the stack size to solve it. 512k is usually enough but we encountered fail on ppc64 even with 2m, therefore the 8m. The build is single threaded so it

Re: [Freeipa-devel] [PATCHES] 0322-0327 New permissions system

2013-12-13 Thread Petr Viktorin
On 12/13/2013 10:49 AM, Martin Kosek wrote: On 12/12/2013 05:17 PM, Petr Viktorin wrote: On 12/12/2013 02:00 PM, Martin Kosek wrote: On 12/06/2013 11:49 AM, Petr Viktorin wrote: On 12/02/2013 01:04 PM, Martin Kosek wrote: On 12/01/2013 11:46 PM, Petr Viktorin wrote: This seems to work now.

Re: [Freeipa-devel] [PATCHES] 0322-0327 New permissions system

2013-12-13 Thread Martin Kosek
On 12/13/2013 01:35 PM, Petr Viktorin wrote: On 12/13/2013 10:49 AM, Martin Kosek wrote: On 12/12/2013 05:17 PM, Petr Viktorin wrote: On 12/12/2013 02:00 PM, Martin Kosek wrote: On 12/06/2013 11:49 AM, Petr Viktorin wrote: On 12/02/2013 01:04 PM, Martin Kosek wrote: On 12/01/2013 11:46 PM,

Re: [Freeipa-devel] [PATCHES] 225-230 Drop support for the legacy LDAP API

2013-12-13 Thread Petr Viktorin
On 12/10/2013 04:05 PM, Jan Cholasta wrote: Hi, I believe the time has come to drop support for the legacy (dn, entry_attrs) tuple API and move to the new LDAPEntry API exclusively. The attached patches convert existing code which uses the old API to the new API and remove backward

Re: [Freeipa-devel] ou, st, l missing from organizationalPerson (Was: FreeIPA 3.3.latest failing tests: config_mod)

2013-12-13 Thread Rich Megginson
On 12/13/2013 02:45 AM, Petr Viktorin wrote: I finally got to investigating this failure. It seems ou, along with a bunch of other attributes, is missing from organizationalPerson in Fedora 20. I don't think it's IPA's fault, as we don't define organizationalPerson. Nathan, could it be

Re: [Freeipa-devel] [PATCH] 531 Increase stack size for Web UI builder

2013-12-13 Thread Martin Kosek
On 12/13/2013 01:18 PM, Alexander Bokovoy wrote: On Fri, 13 Dec 2013, Petr Vobornik wrote: Web UI build fails on some architectures or configuration due to StackOverflow. This patch increases the stack size to solve it. 512k is usually enough but we encountered fail on ppc64 even with 2m,

[Freeipa-devel] [PATCH] 0347 Remove default from the ipapermlocation option

2013-12-13 Thread Petr Viktorin
Fix an oversight in the recent IPA patches that fails a build check on other machines than mine. With required=False and autofill=False, the default is not used. I re-ran the tests to be sure. I'm tempted to ignore the generated API.txt and push it as a one-liner... -- PetrĀ³ From

Re: [Freeipa-devel] [PATCH] 0347 Remove default from the ipapermlocation option

2013-12-13 Thread Petr Viktorin
On 12/13/2013 04:30 PM, Petr Viktorin wrote: Fix an oversight in the recent IPA patches that fails a build check on other machines than mine. With required=False and autofill=False, the default is not used. I re-ran the tests to be sure. I'm tempted to ignore the generated API.txt and push it

Re: [Freeipa-devel] [PATCH 0206] Publish zones only after all LDAP events have been processed

2013-12-13 Thread Petr Spacek
On 12.11.2013 16:08, Petr Spacek wrote: Hello, Publish zones only after all LDAP events have been processed. Zones are not exposed in _default DNS view until all events generated before LDAP intermediate message have been processed. This prevents BIND from returning NXDOMAIN for some names

Re: [Freeipa-devel] [PATCH 0181] Replace LDAP persistent search with syncrepl (RFC 4533)

2013-12-13 Thread Petr Spacek
On 7.10.2013 15:19, Tomas Hozza wrote: On 07/22/2013 03:16 PM, Petr Spacek wrote: On 22.7.2013 13:23, Petr Spacek wrote: Hello, Replace LDAP persistent search with syncrepl (RFC 4533). All direct operations with LDAP Persistent Search control are replaced by ldap_sync_* calls. Syncrepl code

Re: [Freeipa-devel] [PATCH 0201] Report error if RFC 4533 initialization failed

2013-12-13 Thread Petr Spacek
On 24.10.2013 16:00, Tomas Hozza wrote: On 10/23/2013 05:14 PM, Petr Spacek wrote: Hello, this patch belongs to 4.0 release. It allows the user to catch some mis-configurations. It produces error messages like this: LDAP error: Critical extension is unavailable: unable to start SyncRepl

[Freeipa-devel] [PATCH 0210] Add missing default branches to switch statemets

2013-12-13 Thread Petr Spacek
Hello, Add missing default branches to switch statemets. This should help little bit with uninitialized memory usage. This patch should go to branches v3 and master. -- Petr^2 Spacek From c6e2339e1cb6aefe1ad7899eccc416d746e67f3e Mon Sep 17 00:00:00 2001 From: Petr Spacek pspa...@redhat.com

[Freeipa-devel] [PATCH 0208] Remove local variables which shadow variables from a upper level

2013-12-13 Thread Petr Spacek
Hello, Remove local variables which shadow variables from a upper level. This patch should go to branches v3 and master. -- Petr^2 Spacek From 92bf6a1b7808ffe779dbeed18ef845b18f4927fe Mon Sep 17 00:00:00 2001 From: Petr Spacek pspa...@redhat.com Date: Fri, 6 Dec 2013 14:29:16 +0100 Subject:

Re: [Freeipa-devel] [PATCH 0205] Fix race condition during write to internal RBTDB

2013-12-13 Thread Petr Spacek
On 11.11.2013 13:03, Petr Spacek wrote: Hello, Fix race condition during write to internal RBTDB. RBTDB implementation allows to open only one RBTDB instance for writing at the same time. This patch adds mutex to newversion() implementation in ldap_driver.c. See comments around ldapdb_t,

Re: [Freeipa-devel] [PATCH 0197-0200] Preparation for bind-dyndb-ldap release 4.0

2013-12-13 Thread Petr Spacek
On 23.10.2013 17:12, Tomas Hozza wrote: On 10/11/2013 03:35 PM, Petr Spacek wrote: Hello, update documentation and schema files for upcoming version 4.0. This fixes typo in schema file: https://fedorahosted.org/bind-dyndb-ldap/ticket/121 Have a nice weekend! I updated NEWS file in patch

[Freeipa-devel] [PATCH 0214] Make ldap_parse_rrentry() idempotent

2013-12-13 Thread Petr Spacek
Hello, Make ldap_parse_rrentry() idempotent. Now, a call to ldap_parse_rrentry() resets the internal entry interators in ldap_entry_t so the results are always correct. Without this patch, a second call returned empty ldapdb_rdatalist_t because all iterators were at the end of internal lists.

[Freeipa-devel] [PATCH 0216] Bump NVR to 3.6

2013-12-13 Thread Petr Spacek
Hello, Bump NVR to 3.6. BIND 9.9.0 is required. Tomas, shouldn't I use Requires: bind = 32:9.9.0-1 ? This patch should go to branches v3 and master. -- Petr^2 Spacek From d2777beec08e797ff6342d36fc15c480a4f9cee0 Mon Sep 17 00:00:00 2001 From: Petr Spacek pspa...@redhat.com Date: Thu, 12 Dec

[Freeipa-devel] [PATCH 0215] Update NEWS for upcoming 3.6 release

2013-12-13 Thread Petr Spacek
Hello, Update NEWS for upcoming 3.6 release. This patch should go to branches v3 and master. -- Petr^2 Spacek From 72b4e9ea86b9621792cf17292fcb3b7ea90ee399 Mon Sep 17 00:00:00 2001 From: Petr Spacek pspa...@redhat.com Date: Thu, 12 Dec 2013 12:53:46 +0100 Subject: [PATCH] Update NEWS for

Re: [Freeipa-devel] [PATCH 0204] Remove obsolete zr_get_rbt() function from zone register

2013-12-13 Thread Petr Spacek
On 11.11.2013 12:53, Petr Spacek wrote: Hello, Remove obsolete zr_get_rbt() function from zone register. This patch stays unchanged. It should go to branches v3 and master. -- Petr^2 Spacek ___ Freeipa-devel mailing list Freeipa-devel@redhat.com

[Freeipa-devel] [PATCH 0209] Silence GCC warnings produced by -Wjump-misses-init

2013-12-13 Thread Petr Spacek
Hello, Silence GCC warnings produced by -Wjump-misses-init. It seems that it is false alarm in our case. This patch should go to branches v3 and master. -- Petr^2 Spacek From ba457ca6c37b40d59194ea723285ec4b2a7645b0 Mon Sep 17 00:00:00 2001 From: Petr Spacek pspa...@redhat.com Date: Fri, 6

[Freeipa-devel] [PATCH 0212] Remove unused parameter attrlist from ldap_entry_nextattr()

2013-12-13 Thread Petr Spacek
Hello, Remove unused parameter attrlist from ldap_entry_nextattr(). This patch should go to branches v3 and master. -- Petr^2 Spacek From f92881a2e0fa32e7f84816b10984f1e6745d0176 Mon Sep 17 00:00:00 2001 From: Petr Spacek pspa...@redhat.com Date: Wed, 11 Dec 2013 12:10:50 +0100 Subject:

Re: [Freeipa-devel] [PATCH 0207] Do not load invalid zones

2013-12-13 Thread Petr Spacek
On 27.11.2013 16:34, Petr Spacek wrote: Hello, Do not load invalid zones. Without this patch, it was possible to load an invalid zone without proper SOA or NS records because the fake SOA and NS records allowed checks in dns_zone_load() to pass. With this patch, no fake SOA or NS records are

Re: [Freeipa-devel] [PATCH 0202-0203] Improve performance of initial LDAP synchronizationDetect end of initial LDAP synchronization phase

2013-12-13 Thread Petr Spacek
On 12.11.2013 16:13, Petr Spacek wrote: On 5.11.2013 12:29, Tomas Hozza wrote: - Original Message - Hello, Improve performance of initial LDAP synchronization. Changes are not journaled and SOA serial is not incremented during initial LDAP synchronization. This eliminates

[Freeipa-devel] [PATCH 0213] Fix crash caused by invalid data in SOA record

2013-12-13 Thread Petr Spacek
Hello, Fix crash caused by invalid data in SOA record. E.g. try to put '\0' to the idnsSOAmName attribute... This patch should go to branches v3 and master. -- Petr^2 Spacek From b49bea1d803d6e2deb285ad13a84442512f25394 Mon Sep 17 00:00:00 2001 From: Petr Spacek pspa...@redhat.com Date: Fri,

[Freeipa-devel] [PATCH 0217] Cleanup zone and journal files on LDAP reconnect

2013-12-13 Thread Petr Spacek
Hello, Cleanup zone and journal files on LDAP reconnect. This cleanup solves potential inconsistencies between order of operations in LDAP and order of operations recorded in journal. This patch should go to master branch. -- Petr^2 Spacek From f14f2671c3b4f477a64ff06bc282fcd7810b4bda Mon

[Freeipa-devel] Repository with bind-dyndb-ldap 3.6 and 4.0

2013-12-13 Thread Petr Spacek
Hello, latest patches for v3 branch are in my temporary v3.6 branch on Github: https://github.com/spacekpe/bind-dyndb-ldap/tree/v3.6 ... and latest patches for master branch are in master.rbtdb.v22 branch: https://github.com/spacekpe/bind-dyndb-ldap/tree/master.rbtdb.v22 I hope that this will

Re: [Freeipa-devel] [PATCH 0024] Add OTP support to ipalib CLI

2013-12-13 Thread Nathaniel McCallum
On Wed, 2013-12-11 at 13:31 +0100, Martin Kosek wrote: On 12/11/2013 01:24 PM, Jan Cholasta wrote: On 14.11.2013 20:23, Nathaniel McCallum wrote: On Wed, 2013-10-30 at 08:57 +0100, Jan Cholasta wrote: On 8.10.2013 16:35, Nathaniel McCallum wrote: On Tue, 2013-10-08 at 09:19 +0200, Jan

Re: [Freeipa-devel] [PATCH 0024] Add OTP support to ipalib CLI

2013-12-13 Thread Nathaniel McCallum
On Wed, 2013-12-11 at 13:24 +0100, Jan Cholasta wrote: On 14.11.2013 20:23, Nathaniel McCallum wrote: On Wed, 2013-10-30 at 08:57 +0100, Jan Cholasta wrote: On 8.10.2013 16:35, Nathaniel McCallum wrote: On Tue, 2013-10-08 at 09:19 +0200, Jan Cholasta wrote: +class

Re: [Freeipa-devel] [PATCH 0024] Add OTP support to ipalib CLI

2013-12-13 Thread Nathaniel McCallum
On Fri, 2013-12-13 at 14:50 -0500, Nathaniel McCallum wrote: On Wed, 2013-12-11 at 13:24 +0100, Jan Cholasta wrote: On 14.11.2013 20:23, Nathaniel McCallum wrote: On Wed, 2013-10-30 at 08:57 +0100, Jan Cholasta wrote: On 8.10.2013 16:35, Nathaniel McCallum wrote: On Tue, 2013-10-08 at

Re: [Freeipa-devel] FreeIPA OTP End-to-End

2013-12-13 Thread Dmitri Pal
On 12/13/2013 03:57 PM, Nathaniel McCallum wrote: This is an email to track the status of the OTP project as we push toward completion. I'm also attempting to get all the pieces in play so that they are testable. RPMs Available here: http://npmccallum.fedorapeople.org/freeipa-otp/rpms/