Re: [Freeipa-devel] [PATCHES] 152-158 ipa-server-certinstall fixes

2013-08-20 Thread Jan Cholasta
On 19.8.2013 17:53, Petr Viktorin wrote: On 08/19/2013 03:50 PM, Jan Cholasta wrote: On 19.8.2013 14:02, Petr Viktorin wrote: Thanks! I've read the patches and have some initial comments; I'll get to functional testing (and writing related CA-less tests) right away. The patches need a small

Re: [Freeipa-devel] [PATCH] 0056 Fix broken replication

2013-08-20 Thread Petr Viktorin
On 08/19/2013 06:16 PM, Ana Krivokapic wrote: On 08/19/2013 06:01 PM, Petr Viktorin wrote: On 08/19/2013 05:50 PM, Ana Krivokapic wrote: Hello, This patch addresses tickethttps://fedorahosted.org/freeipa/ticket/3868. -- Regards, Ana Krivokapic Associate Software Engineer FreeIPA team Red Hat

Re: [Freeipa-devel] [PATCH] 0056 Fix broken replication

2013-08-20 Thread Ana Krivokapic
On 08/20/2013 01:17 PM, Petr Viktorin wrote: On 08/19/2013 06:16 PM, Ana Krivokapic wrote: On 08/19/2013 06:01 PM, Petr Viktorin wrote: On 08/19/2013 05:50 PM, Ana Krivokapic wrote: Hello, This patch addresses tickethttps://fedorahosted.org/freeipa/ticket/3868. -- Regards, Ana Krivokapic

Re: [Freeipa-devel] [PATCH] 445 Web UI integration tests: ID range types

2013-08-20 Thread Petr Vobornik
On 08/19/2013 05:33 PM, Petr Vobornik wrote: https://fedorahosted.org/freeipa/ticket/3834 New version. Fixes usage of secondary rid for non-trust installation. -- Petr Vobornik From 4f8358e9aa8a4ec2e28d47acd351b556cd10c272 Mon Sep 17 00:00:00 2001 From: Petr Vobornik pvobo...@redhat.com

Re: [Freeipa-devel] [PATCHES] 152-158 ipa-server-certinstall fixes

2013-08-20 Thread Petr Viktorin
On 08/20/2013 09:10 AM, Jan Cholasta wrote: On 19.8.2013 17:53, Petr Viktorin wrote: On 08/19/2013 03:50 PM, Jan Cholasta wrote: On 19.8.2013 14:02, Petr Viktorin wrote: Thanks! I've read the patches and have some initial comments; I'll get to functional testing (and writing related CA-less

Re: [Freeipa-devel] [PATCH 0091] Perform dirsrv tuning at platform level

2013-08-20 Thread Nathaniel McCallum
On Mon, 2013-08-19 at 14:48 +0200, Tomas Babej wrote: Hi, When configuring the 389 Directory Server instance, we tune it so that number of file descriptors available to the DS is increased from the default 1024 to 8192. There are platform specific steps that need to be conducted

Re: [Freeipa-devel] [PATCH] 443 Web UI integration tests: CA-less

2013-08-20 Thread Ana Krivokapic
On 08/15/2013 04:03 PM, Petr Vobornik wrote: +if is_visible: +is_enabled = 'disabled' not in link.get_attribute(class).split(' ') Nitpick: it would be better to use .split() without arguments here. Here's an example that illustrates the difference in behavior between