Re: [Freeipa-devel] Splitting out ipaldap

2015-04-14 Thread Jan Cholasta
Dne 14.4.2015 v 19:21 Petr Viktorin napsal(a): On 04/14/2015 06:18 PM, Jan Cholasta wrote: Dne 14.4.2015 v 17:50 Petr Viktorin napsal(a): On 04/14/2015 05:22 PM, Jan Cholasta wrote: Hi, Dne 14.4.2015 v 16:38 Petr Viktorin napsal(a): Hello! As some of you know, I'm looking to help porting Fr

Re: [Freeipa-devel] [PATCH 0025] proper client host setup/teardown in forced client reenrollment integration test suite

2015-04-14 Thread Petr Vobornik
On 04/14/2015 03:54 PM, Milan Kubik wrote: On 04/14/2015 03:20 PM, Milan Kubik wrote: On 03/31/2015 10:42 AM, Martin Babinsky wrote: During the investigation of https://fedorahosted.org/freeipa/ticket/4614 I discovered a bug (?) in forced client reenrollment integration test. During test s

Re: [Freeipa-devel] [PATCH] 811 performance: faster DN implementation

2015-04-14 Thread Petr Vobornik
On 04/14/2015 09:32 AM, Petr Spacek wrote: On 10.4.2015 13:11, Petr Viktorin wrote: ACK Pushed to master: 11bd9d96f191066f7ba760549f00179c128a9787 Please be so kind and fix naming. AFAIK the patch refers to 'openldap' DN format but in fact it is python-ldap-ishm. It will surely confuse next

Re: [Freeipa-devel] [PATCH 0210] DNSSEC: CI test

2015-04-14 Thread Petr Vobornik
On 04/13/2015 04:07 PM, Milan Kubik wrote: On 04/08/2015 12:46 PM, Martin Basti wrote: On 07/04/15 15:45, Milan Kubik wrote: updated patch attached. -- Martin Basti Thanks, ack. Milan Pushed to: master: 0a1a3d73120bdf20ae05bcf663f14ca1a8b02c25 ipa-4-1: f3b5d163bf7432d1a89e7798a3192968bd0

Re: [Freeipa-devel] [PATCHES 0213 - 0221] Server Upgrade: LDAPI, Update plugins

2015-04-14 Thread Petr Vobornik
On 04/10/2015 02:19 PM, David Kupka wrote: On 04/01/2015 05:22 PM, Martin Basti wrote: On 24/03/15 12:08, Martin Basti wrote: On 24/03/15 09:56, Martin Basti wrote: On 23/03/15 15:47, Martin Basti wrote: Hello, The patches: * allows to specify order of update plugins in update files. * requi

Re: [Freeipa-devel] Splitting out ipaldap

2015-04-14 Thread Petr Viktorin
On 04/14/2015 06:18 PM, Jan Cholasta wrote: Dne 14.4.2015 v 17:50 Petr Viktorin napsal(a): On 04/14/2015 05:22 PM, Jan Cholasta wrote: Hi, Dne 14.4.2015 v 16:38 Petr Viktorin napsal(a): Hello! As some of you know, I'm looking to help porting FreeIPA to Python 3. One of the major dependencies

Re: [Freeipa-devel] [PATCH 0045] Add message for skipping NTP configuration during client install

2015-04-14 Thread Petr Vobornik
On 04/07/2015 05:46 PM, Martin Basti wrote: On 07/04/15 17:06, Gabe Alford wrote: Stupid me. I realized that chronyd was running which messed up my testing and such (sorry about that). New patch attached that implements 'else' On Tue, Apr 7, 2015 at 2:32 AM, Martin Basti mailto:mba...@redhat.co

Re: [Freeipa-devel] [PATCH] webui: use no_members option in entity select search

2015-04-14 Thread Petr Vobornik
On 04/01/2015 03:01 PM, Martin Babinsky wrote: On 03/31/2015 04:16 PM, Petr Vobornik wrote: Obtaining member information for entity selects is not needed and it causes unwanted performance hit, especially with larger groups. This patch removes it. https://fedorahosted.org/freeipa/ticket/4948

Re: [Freeipa-devel] [PATCH] 786 webui: unable to select single value in CB by enter key

2015-04-14 Thread Petr Vobornik
On 03/31/2015 06:07 PM, Martin Babinsky wrote: On 03/31/2015 04:19 PM, Petr Vobornik wrote: This little fellow was hiding in a cupboard (patchset 784-786 was abandoned). Fix: If editable combobox has one value, the value is selected and changed by hand, it can't be re-selected by enter key.

Re: [Freeipa-devel] Splitting out ipaldap

2015-04-14 Thread Petr Viktorin
On 04/14/2015 06:03 PM, Rob Crittenden wrote: Petr Viktorin wrote: On 04/14/2015 05:19 PM, John Dennis wrote: On 04/14/2015 10:38 AM, Petr Viktorin wrote: Hello! As some of you know, I'm looking to help porting FreeIPA to Python 3. One of the major dependencies holding this back is python-lda

Re: [Freeipa-devel] Splitting out ipaldap

2015-04-14 Thread Jan Cholasta
Dne 14.4.2015 v 17:50 Petr Viktorin napsal(a): On 04/14/2015 05:22 PM, Jan Cholasta wrote: Hi, Dne 14.4.2015 v 16:38 Petr Viktorin napsal(a): Hello! As some of you know, I'm looking to help porting FreeIPA to Python 3. One of the major dependencies holding this back is python-ldap, which hasn

Re: [Freeipa-devel] Splitting out ipaldap

2015-04-14 Thread Rob Crittenden
Petr Viktorin wrote: > On 04/14/2015 05:19 PM, John Dennis wrote: >> On 04/14/2015 10:38 AM, Petr Viktorin wrote: >>> Hello! >>> >>> As some of you know, I'm looking to help porting FreeIPA to Python 3. >>> One of the major dependencies holding this back is python-ldap, which >>> hasn't been ported

Re: [Freeipa-devel] Splitting out ipaldap

2015-04-14 Thread Petr Viktorin
On 04/14/2015 05:22 PM, Jan Cholasta wrote: Hi, Dne 14.4.2015 v 16:38 Petr Viktorin napsal(a): Hello! As some of you know, I'm looking to help porting FreeIPA to Python 3. One of the major dependencies holding this back is python-ldap, which hasn't been ported yet. Some preliminary porting pat

Re: [Freeipa-devel] Splitting out ipaldap

2015-04-14 Thread Petr Viktorin
On 04/14/2015 05:19 PM, John Dennis wrote: On 04/14/2015 10:38 AM, Petr Viktorin wrote: Hello! As some of you know, I'm looking to help porting FreeIPA to Python 3. One of the major dependencies holding this back is python-ldap, which hasn't been ported yet. Some preliminary porting patches by

Re: [Freeipa-devel] Splitting out ipaldap

2015-04-14 Thread Jan Cholasta
Hi, Dne 14.4.2015 v 16:38 Petr Viktorin napsal(a): Hello! As some of you know, I'm looking to help porting FreeIPA to Python 3. One of the major dependencies holding this back is python-ldap, which hasn't been ported yet. Some preliminary porting patches by Raphaël Barrois [0] are ready and hav

Re: [Freeipa-devel] Splitting out ipaldap

2015-04-14 Thread John Dennis
On 04/14/2015 10:38 AM, Petr Viktorin wrote: > Hello! > > As some of you know, I'm looking to help porting FreeIPA to Python 3. > One of the major dependencies holding this back is python-ldap, which > hasn't been ported yet. Some preliminary porting patches by Raphaël > Barrois [0] are ready an

[Freeipa-devel] Splitting out ipaldap

2015-04-14 Thread Petr Viktorin
Hello! As some of you know, I'm looking to help porting FreeIPA to Python 3. One of the major dependencies holding this back is python-ldap, which hasn't been ported yet. Some preliminary porting patches by Raphaël Barrois [0] are ready and have been sent to the python-ldap list. The python-ld

Re: [Freeipa-devel] [PATCH 0029] suppress errors arising from deleting non-existent files during client uninstall

2015-04-14 Thread Martin Basti
On 14/04/15 16:12, Martin Basti wrote: On 14/04/15 14:25, Martin Babinsky wrote: This patch addresses https://fedorahosted.org/freeipa/ticket/4966 The noise during rollback/uninstall is caused mainly by unsuccessful attempts to remove files that do not exist anymore. These errors are now logg

Re: [Freeipa-devel] [PATCH 0029] suppress errors arising from deleting non-existent files during client uninstall

2015-04-14 Thread Martin Basti
On 14/04/15 14:25, Martin Babinsky wrote: This patch addresses https://fedorahosted.org/freeipa/ticket/4966 The noise during rollback/uninstall is caused mainly by unsuccessful attempts to remove files that do not exist anymore. These errors are now logged at debug level and do not pop-up to s

Re: [Freeipa-devel] [PATCH 0025] proper client host setup/teardown in forced client reenrollment integration test suite

2015-04-14 Thread Milan Kubik
On 04/14/2015 03:20 PM, Milan Kubik wrote: On 03/31/2015 10:42 AM, Martin Babinsky wrote: During the investigation of https://fedorahosted.org/freeipa/ticket/4614 I discovered a bug (?) in forced client reenrollment integration test. During test scenario, master and replica are setup corr

Re: [Freeipa-devel] [PATCH 0027] do not install CA on replica during integration test if setup_ca=False

2015-04-14 Thread Milan Kubik
On 04/08/2015 08:44 AM, Martin Babinsky wrote: I have discovered another little bug in the integration test suite. Attaching a patch that fixes it. Hello, thanks for the patch. I hereby invoke the "One Liner" rule. Cheers, Milan -- Manage your subscription for the Freeipa-devel maili

Re: [Freeipa-devel] [PATCH 0025] proper client host setup/teardown in forced client reenrollment integration test suite

2015-04-14 Thread Milan Kubik
On 03/31/2015 10:42 AM, Martin Babinsky wrote: During the investigation of https://fedorahosted.org/freeipa/ticket/4614 I discovered a bug (?) in forced client reenrollment integration test. During test scenario, master and replica are setup correctly at the beginning of the test, but the c

Re: [Freeipa-devel] [PATCH] manage replication topology in the shared tree

2015-04-14 Thread Ludwig Krispenz
Hi Martin, thanks for your time and comments. The design page should match the current implemtation, but as you said there were some iterations and I will check again and address the points you raised. Ludwig On 04/14/2015 01:39 PM, Martin Kosek wrote: On 04/13/2015 10:56 AM, Ludwig Krispen

[Freeipa-devel] [PATCH 0029] suppress errors arising from deleting non-existent files during client uninstall

2015-04-14 Thread Martin Babinsky
This patch addresses https://fedorahosted.org/freeipa/ticket/4966 The noise during rollback/uninstall is caused mainly by unsuccessful attempts to remove files that do not exist anymore. These errors are now logged at debug level and do not pop-up to stdout/stderr. -- Martin^3 Babinsky From d

Re: [Freeipa-devel] [PATCH] manage replication topology in the shared tree

2015-04-14 Thread thierry bordaz
On 04/14/2015 01:39 PM, Martin Kosek wrote: On 04/13/2015 10:56 AM, Ludwig Krispenz wrote: Hi, in the attachment you find the latest state of the "topology plugin", it implements what is defined in the design page: http://www.freeipa.org/page/V4/Manage_replication_topology (which is also waitin

Re: [Freeipa-devel] [PATCH] manage replication topology in the shared tree

2015-04-14 Thread Martin Kosek
On 04/13/2015 10:56 AM, Ludwig Krispenz wrote: > Hi, > > in the attachment you find the latest state of the "topology plugin", it > implements what is defined in the design page: > http://www.freeipa.org/page/V4/Manage_replication_topology (which is also > waiting for a reviewer) > > It contains

Re: [Freeipa-devel] [PATCH] 0004 User life cycle: support of MODRDN to a new superior

2015-04-14 Thread Jan Cholasta
Dne 12.4.2015 v 18:51 thierry bordaz napsal(a): On 04/09/2015 12:42 PM, thierry bordaz wrote: On 04/08/2015 03:33 PM, Jan Cholasta wrote: Dne 8.4.2015 v 15:00 thierry bordaz napsal(a): On 04/08/2015 08:34 AM, Jan Cholasta wrote: Hi, Dne 1.4.2015 v 17:40 thierry bordaz napsal(a): Hello,

Re: [Freeipa-devel] [PATCH 0026-0028] Fix nits in user-visible output

2015-04-14 Thread Petr Spacek
On 14.4.2015 09:10, Martin Kosek wrote: > On 04/13/2015 05:05 PM, Petr Spacek wrote: >> Hello, >> >> documentation team proposed few changes in user-visible messages so here it >> is. It was not worth a ticket and related overhead. > > The changes look OK to me. I would just have one (prudish) req

Re: [Freeipa-devel] [PATCH] 811 performance: faster DN implementation

2015-04-14 Thread Petr Spacek
On 10.4.2015 13:11, Petr Viktorin wrote: > ACK Please be so kind and fix naming. AFAIK the patch refers to 'openldap' DN format but in fact it is python-ldap-ishm. It will surely confuse next generation of FreeIPA developers :-) -- Petr^2 Spacek -- Manage your subscription for the Freeipa-deve

Re: [Freeipa-devel] [PATCH 0026-0028] Fix nits in user-visible output

2015-04-14 Thread Martin Kosek
On 04/13/2015 05:05 PM, Petr Spacek wrote: > Hello, > > documentation team proposed few changes in user-visible messages so here it > is. It was not worth a ticket and related overhead. The changes look OK to me. I would just have one (prudish) request to not add nazi reference to our git history