Re: [Freeipa-devel] [PATCH 0064] destroy httpd ccache after stopping the service

2015-09-23 Thread Martin Basti
On 09/22/2015 01:44 PM, Martin Babinsky wrote: This patch fixes https://fedorahosted.org/freeipa/ticket/5296 and generally makes cleaning up of httpd ccache more thorough. ACK Pushed to: master: 93d080d726359db16749104c8bc20d14a5455dc0 ipa-4-2: 23f1d4ed605f9f4b22d6ad93c4110fff7358682c --

Re: [Freeipa-devel] [PATCH 494] install: create kdcproxy user during server install

2015-09-23 Thread Christian Heimes
On 2015-09-23 12:40, Jan Cholasta wrote: > On 23.9.2015 11:44, Christian Heimes wrote: >> On 2015-09-23 10:54, Jan Cholasta wrote: Correction, the HTTP server works, but it spits lots of errors in error_log about /var/lib/kdcproxy not existing. Is the KDCProxy supposed to be

Re: [Freeipa-devel] [PATCH 0065] CI test for IPA install/backup/uninstall/install/restore scenario

2015-09-23 Thread Martin Babinsky
On 09/23/2015 12:53 PM, Martin Babinsky wrote: CI test for full IPA restore into a running IPA server self-NACK -- Martin^3 Babinsky -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

Re: [Freeipa-devel] [PATCH 494] install: create kdcproxy user during server install

2015-09-23 Thread Jan Cholasta
On 23.9.2015 11:44, Christian Heimes wrote: On 2015-09-23 10:54, Jan Cholasta wrote: Correction, the HTTP server works, but it spits lots of errors in error_log about /var/lib/kdcproxy not existing. Is the KDCProxy supposed to be installked/enabled on upgrade ? If not, why not ? Even if it is

Re: [Freeipa-devel] [PATCHES 362-366] Realmdomains handling improvements

2015-09-23 Thread Martin Basti
On 09/22/2015 02:23 PM, Tomas Babej wrote: On 09/03/2015 04:34 PM, Alexander Bokovoy wrote: On Thu, 03 Sep 2015, Tomas Babej wrote: Hi, this couple of patches fix https://fedorahosted.org/freeipa/ticket/5278 and improve our handling of realmdomains in general. The code looks good to me. I

[Freeipa-devel] [PATCHES 0370-0371] winsync-migrate: Handle invalid characters in the names of IPA entities

2015-09-23 Thread Tomas Babej
Hi, this fixes https://fedorahosted.org/freeipa/ticket/5319. Details in the commit messages. Tomas From 7daf849d776696575f9ce7a374995845616341b8 Mon Sep 17 00:00:00 2001 From: Tomas Babej Date: Wed, 23 Sep 2015 13:27:35 +0200 Subject: [PATCH] winsync-migrate: Convert entity

[Freeipa-devel] [PATCH 0065] CI test for IPA install/backup/uninstall/install/restore scenario

2015-09-23 Thread Martin Babinsky
Should help to catch bugs like https://fedorahosted.org/freeipa/ticket/5296 -- Martin^3 Babinsky From e515bc3aff47ac83807f2ae0b625e0ef8291b7c9 Mon Sep 17 00:00:00 2001 From: Martin Babinsky Date: Mon, 21 Sep 2015 09:58:38 +0200 Subject: [PATCH] CI test for full IPA restore

Re: [Freeipa-devel] [PATCH 494] install: create kdcproxy user during server install

2015-09-23 Thread Jan Cholasta
On 23.9.2015 12:49, Christian Heimes wrote: On 2015-09-23 12:40, Jan Cholasta wrote: On 23.9.2015 11:44, Christian Heimes wrote: On 2015-09-23 10:54, Jan Cholasta wrote: Correction, the HTTP server works, but it spits lots of errors in error_log about /var/lib/kdcproxy not existing. Is the

Re: [Freeipa-devel] [PATCH 0055] dnssec options missing in ipa-dns-install man page

2015-09-23 Thread Gabe Alford
Odd and done. Updated patch attached. Gabe On Wed, Sep 23, 2015 at 5:20 AM, Martin Basti wrote: > > > On 09/22/2015 03:32 PM, Gabe Alford wrote: > >> create mode 100644 >> install/tools/man/freeipa-rga-0055-dnssec-options-missing-in-ipa-dns-install-man-page.patch >> > Hello,

Re: [Freeipa-devel] [PATCH 0055] dnssec options missing in ipa-dns-install man page

2015-09-23 Thread Martin Basti
On 09/23/2015 03:12 PM, Gabe Alford wrote: Odd and done. Updated patch attached. Gabe On Wed, Sep 23, 2015 at 5:20 AM, Martin Basti > wrote: On 09/22/2015 03:32 PM, Gabe Alford wrote: create mode 100644

Re: [Freeipa-devel] [PATCH 0055] dnssec options missing in ipa-dns-install man page

2015-09-23 Thread Gabe Alford
Thanks. Updated patch attached. On Wed, Sep 23, 2015 at 7:14 AM, Martin Basti wrote: > > > On 09/23/2015 03:12 PM, Gabe Alford wrote: > > Odd and done. Updated patch attached. > > Gabe > > On Wed, Sep 23, 2015 at 5:20 AM, Martin Basti wrote: > >> >> >> On

Re: [Freeipa-devel] [PATCH 494] install: create kdcproxy user during server install

2015-09-23 Thread Simo Sorce
On Wed, 2015-09-23 at 13:37 +0200, Jan Cholasta wrote: > On 23.9.2015 12:49, Christian Heimes wrote: > > On 2015-09-23 12:40, Jan Cholasta wrote: > >> On 23.9.2015 11:44, Christian Heimes wrote: > >>> On 2015-09-23 10:54, Jan Cholasta wrote: > > Correction, the HTTP server works, but it spits

Re: [Freeipa-devel] [PATCHES 0370-0371] winsync-migrate: Handle invalid characters in the names of IPA entities

2015-09-23 Thread Jan Cholasta
On 23.9.2015 17:03, Martin Babinsky wrote: On 09/23/2015 04:56 PM, Jan Cholasta wrote: On 23.9.2015 16:52, Martin Babinsky wrote: On 09/23/2015 01:39 PM, Tomas Babej wrote: Hi, this fixes https://fedorahosted.org/freeipa/ticket/5319. Details in the commit messages. Tomas ACK The

Re: [Freeipa-devel] [PATCHES 0370-0371] winsync-migrate: Handle invalid characters in the names of IPA entities

2015-09-23 Thread Jan Cholasta
On 23.9.2015 16:52, Martin Babinsky wrote: On 09/23/2015 01:39 PM, Tomas Babej wrote: Hi, this fixes https://fedorahosted.org/freeipa/ticket/5319. Details in the commit messages. Tomas ACK The patches need to be rebased on top of ipa-4-2. -- Jan Cholasta -- Manage your subscription

Re: [Freeipa-devel] [PATCHES 0370-0371] winsync-migrate: Handle invalid characters in the names of IPA entities

2015-09-23 Thread Martin Babinsky
On 09/23/2015 04:56 PM, Jan Cholasta wrote: On 23.9.2015 16:52, Martin Babinsky wrote: On 09/23/2015 01:39 PM, Tomas Babej wrote: Hi, this fixes https://fedorahosted.org/freeipa/ticket/5319. Details in the commit messages. Tomas ACK The patches need to be rebased on top of ipa-4-2.

Re: [Freeipa-devel] [PATCHES 0370-0371] winsync-migrate: Handle invalid characters in the names of IPA entities

2015-09-23 Thread Tomas Babej
On 09/23/2015 05:06 PM, Tomas Babej wrote: > > > On 09/23/2015 04:56 PM, Jan Cholasta wrote: >> On 23.9.2015 16:52, Martin Babinsky wrote: >>> On 09/23/2015 01:39 PM, Tomas Babej wrote: Hi, this fixes https://fedorahosted.org/freeipa/ticket/5319. Details in the commit

Re: [Freeipa-devel] [PATCH 494] install: create kdcproxy user during server install

2015-09-23 Thread Martin Babinsky
On 09/23/2015 01:37 PM, Jan Cholasta wrote: On 23.9.2015 12:49, Christian Heimes wrote: On 2015-09-23 12:40, Jan Cholasta wrote: On 23.9.2015 11:44, Christian Heimes wrote: On 2015-09-23 10:54, Jan Cholasta wrote: Correction, the HTTP server works, but it spits lots of errors in error_log

Re: [Freeipa-devel] [PATCHES 0370-0371] winsync-migrate: Handle invalid characters in the names of IPA entities

2015-09-23 Thread Tomas Babej
On 09/23/2015 04:56 PM, Jan Cholasta wrote: > On 23.9.2015 16:52, Martin Babinsky wrote: >> On 09/23/2015 01:39 PM, Tomas Babej wrote: >>> Hi, >>> >>> this fixes https://fedorahosted.org/freeipa/ticket/5319. >>> >>> Details in the commit messages. >>> >>> Tomas >>> >>> >> >> ACK >> > > The

Re: [Freeipa-devel] [PATCH 494] install: create kdcproxy user during server install

2015-09-23 Thread Jan Cholasta
On 23.9.2015 16:25, Martin Babinsky wrote: On 09/23/2015 01:37 PM, Jan Cholasta wrote: On 23.9.2015 12:49, Christian Heimes wrote: On 2015-09-23 12:40, Jan Cholasta wrote: On 23.9.2015 11:44, Christian Heimes wrote: On 2015-09-23 10:54, Jan Cholasta wrote: Correction, the HTTP server works,

Re: [Freeipa-devel] [PATCHES 0370-0371] winsync-migrate: Handle invalid characters in the names of IPA entities

2015-09-23 Thread Martin Babinsky
On 09/23/2015 01:39 PM, Tomas Babej wrote: Hi, this fixes https://fedorahosted.org/freeipa/ticket/5319. Details in the commit messages. Tomas ACK -- Martin^3 Babinsky -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCHES 466-468, 0316] install: Add common base class for server and replica install

2015-09-23 Thread Martin Basti
On 09/22/2015 12:10 PM, Jan Cholasta wrote: On 22.9.2015 10:29, Martin Babinsky wrote: On 09/16/2015 10:44 AM, Jan Cholasta wrote: On 16.9.2015 08:11, Jan Cholasta wrote: On 15.9.2015 07:22, Jan Cholasta wrote: On 10.8.2015 16:58, Martin Babinsky wrote: On 08/06/2015 08:22 AM, Jan

Re: [Freeipa-devel] [PATCHSET] Replica promotion patches

2015-09-23 Thread Simo Sorce
On Wed, 2015-09-23 at 08:35 +0200, Jan Cholasta wrote: > What I mean is that installing a replica using an already existing > replica file should be prevented at level 1 as well: > > root@ipa1# ipa-server-install --domain-level=0 > root@ipa1# ipa-replica-prepare ipa2.example.com > root@ipa1# ipa

Re: [Freeipa-devel] [PATCH] improved logging in dnssec tests

2015-09-23 Thread Petr Spacek
On 22.9.2015 11:21, Oleg Fayans wrote: > Hi all, > > I've noticed that in some tests some low-level functions can return False in a > number of different conditions, which severely complicates test debugging. > This patch implements the approach widely used in the Go language (and maybe, > some

Re: [Freeipa-devel] [PATCH] Proper fix for ticket 5306

2015-09-23 Thread Petr Spacek
On 22.9.2015 10:42, Oleg Fayans wrote: > +++ b/ipatests/test_integration/tasks.py > @@ -58,6 +58,14 @@ def check_arguments_are(slice, instanceof): > return wrapped > return wrapper > > +def prepare_reverse_zone(host, ip): > +nums = ip.split('.')[:-1] > +zone =

Re: [Freeipa-devel] [PATCHSET] Replica promotion patches

2015-09-23 Thread Jan Cholasta
On 23.9.2015 02:47, Simo Sorce wrote: On Tue, 2015-09-22 at 10:57 -0400, Simo Sorce wrote: On Tue, 2015-09-22 at 10:45 +0200, Jan Cholasta wrote: Hi, On 9.9.2015 20:25, Simo Sorce wrote: On Wed, 2015-08-26 at 17:27 -0400, Simo Sorce wrote: This patchset implements

Re: [Freeipa-devel] [PATCH 494] install: create kdcproxy user during server install

2015-09-23 Thread Jan Cholasta
On 23.9.2015 02:22, Simo Sorce wrote: On Tue, 2015-09-22 at 20:09 -0400, Simo Sorce wrote: On Tue, 2015-09-22 at 16:35 +0200, Jan Cholasta wrote: On 22.9.2015 15:11, Martin Babinsky wrote: On 09/22/2015 01:33 PM, Jan Cholasta wrote: Hi, the attached patch fixes

Re: [Freeipa-devel] [PATCH 494] install: create kdcproxy user during server install

2015-09-23 Thread Christian Heimes
On 2015-09-23 10:54, Jan Cholasta wrote: >> Correction, the HTTP server works, but it spits lots of errors in >> error_log about /var/lib/kdcproxy not existing. >> >> Is the KDCProxy supposed to be installked/enabled on upgrade ? >> If not, why not ? >> Even if it is not enabled, shouldn't the

Re: [Freeipa-devel] [PATCHSET] Replica promotion patches

2015-09-23 Thread Martin Basti
On 09/22/2015 10:45 AM, Jan Cholasta wrote: Hi, On 9.9.2015 20:25, Simo Sorce wrote: On Wed, 2015-08-26 at 17:27 -0400, Simo Sorce wrote: This patchset implements https://fedorahosted.org/freeipa/ticket/2888 and introduces a number of required changes and dependencies to achieve this