Re: [Freeipa-devel] [PATCH 515] client install: do not corrupt OpenSSH config with Match sections

2015-11-20 Thread Martin Babinsky
On 11/20/2015 09:56 AM, Jan Cholasta wrote: Hi, the attached patch fixes . Honza ACK -- Martin^3 Babinsky -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to

Re: [Freeipa-devel] [PATCH 0353] upgrade: fix migration of old dns forward zones

2015-11-20 Thread Martin Basti
On 20.11.2015 14:34, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5472 Patch attached. I forgot to attached rebased patch for ipa-4-2 From dac7361785c83bfccc9ae757a081a88c13829fe8 Mon Sep 17 00:00:00 2001 From: Martin Basti Date: Fri, 20 Nov 2015

[Freeipa-devel] [PATCH] 927 topology: treat server suffix as multivalued attribute in API

2015-11-20 Thread Petr Vobornik
-- Petr Vobornik From d4f0851d528628ec25592e901e8ddef18ecbf6e9 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Thu, 29 Oct 2015 19:01:09 +0100 Subject: [PATCH] topology: treat server suffix as multivalued attribute in API --- API.txt| 2 +-

[Freeipa-devel] [PATCH 0354] Remove forgotten print in DNS plugin

2015-11-20 Thread Martin Basti
patch attached. From b916e78014ced8584d27c427f680b96d1ce594ea Mon Sep 17 00:00:00 2001 From: Martin Basti Date: Fri, 20 Nov 2015 14:29:12 +0100 Subject: [PATCH] remove forgotten print in DNS plugin --- ipalib/plugins/dns.py | 1 - 1 file changed, 1 deletion(-) diff --git

[Freeipa-devel] [PATCH 0353] upgrade: fix migration of old dns forward zones

2015-11-20 Thread Martin Basti
https://fedorahosted.org/freeipa/ticket/5472 Patch attached. From dac7361785c83bfccc9ae757a081a88c13829fe8 Mon Sep 17 00:00:00 2001 From: Martin Basti Date: Fri, 20 Nov 2015 11:53:06 +0100 Subject: [PATCH] upgrade: fix migration of old dns forward zones Plugins should call

[Freeipa-devel] [PATCH 0101] raise more descriptive Backend connection-related exceptions

2015-11-20 Thread Martin Babinsky
Fixes https://fedorahosted.org/freeipa/ticket/5473 Patch is for master only, I will rebase it for 4-2 if we decide that it can land there as well. -- Martin^3 Babinsky From 9072fad67fd0bc3b9ef08d59a0c59c949a78e87a Mon Sep 17 00:00:00 2001 From: Martin Babinsky Date: Fri,

Re: [Freeipa-devel] [PATCH 0331, 0337] User plugin: allow multiple managers per user - CLI part

2015-11-20 Thread Martin Basti
On 19.11.2015 14:57, David Kupka wrote: On 19/11/15 12:54, Martin Basti wrote: On 18.11.2015 16:10, Martin Basti wrote: On 12.11.2015 12:39, Martin Basti wrote: On 27.10.2015 14:59, Martin Basti wrote: On 20.10.2015 18:46, Martin Basti wrote: On 20.10.2015 16:07, Martin Basti

[Freeipa-devel] [PATCH 0102] update idrange tests to reflect disabled modification of local ID ranges

2015-11-20 Thread Martin Babinsky
When I fixed https://fedorahosted.org/freeipa/ticket/4826 I forgot to fix the corresponding xmlrpc tests. This oversight bit me today when I ran in-tree tests on my VM. Here is the patch that makes idrange tests green and shiny again. -- Martin^3 Babinsky From

Re: [Freeipa-devel] [PATCH 0332] fix user post_callback

2015-11-20 Thread Martin Basti
On 19.11.2015 14:56, David Kupka wrote: On 12/11/15 12:36, Martin Basti wrote: On 21.10.2015 11:14, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5387 Patch attached. Fix for this ticket has been implemented in patch mbasti-0331-2 Attached patch contains only common

[Freeipa-devel] [PATCH 0064] Check if IPA is configured before attempting a winsync migration

2015-11-20 Thread Gabe Alford
Hello, Fix for https://fedorahosted.org/freeipa/ticket/5470 Thanks, Gabe From 9e9b8813d069b3a65e16ef90a602bf35feade9c9 Mon Sep 17 00:00:00 2001 From: Gabe Date: Fri, 20 Nov 2015 07:54:30 -0700 Subject: [PATCH] Check if IPA is configured before attempting a winsync

Re: [Freeipa-devel] [PATCH 0100] replica promotion: modify default.conf even if DS configuration fails

2015-11-20 Thread Martin Babinsky
On 11/20/2015 10:19 AM, Martin Babinsky wrote: Fixes https://fedorahosted.org/freeipa/ticket/5417 Sorry forgot to add the patch. -- Martin^3 Babinsky From f6235e6b1bcb2dfe08b0d92a9f0b834d45801ab9 Mon Sep 17 00:00:00 2001 From: Martin Babinsky Date: Fri, 20 Nov 2015

Re: [Freeipa-devel] [TESTS][PATCH 0006] Add comments to stageuser plugin tests

2015-11-20 Thread Martin Babinsky
On 11/19/2015 10:34 AM, Petr Viktorin wrote: On 11/19/2015 09:30 AM, Lenka Doudova wrote: On 11/18/2015 04:51 PM, Martin Babinsky wrote: On 11/18/2015 02:16 PM, Lenka Doudova wrote: Hi, here's a patch that adds a few comments to stageuser tests in order to allow easier determining of a

[Freeipa-devel] [PATCH 0002] Refactor test_group_plugin

2015-11-20 Thread Filip Škola
Another one. F. >From 0f0edcb1c7e32e24cf421e197657b8ac3a6a16a8 Mon Sep 17 00:00:00 2001 From: Filip Skola Date: Mon, 9 Nov 2015 16:48:55 +0100 Subject: [PATCH] Refactor test_group_plugin, use GroupTracker for tests --- ipatests/test_xmlrpc/test_group_plugin.py | 1896

Re: [Freeipa-devel] [PATCH 515] client install: do not corrupt OpenSSH config with Match sections

2015-11-20 Thread Jan Cholasta
On 20.11.2015 12:15, Martin Babinsky wrote: On 11/20/2015 09:56 AM, Jan Cholasta wrote: Hi, the attached patch fixes . Honza ACK Thanks. Pushed to: master: 2d041daf17a4c461489cdd0879056d0cb5eacde5 ipa-4-2:

Re: [Freeipa-devel] [PATCH 0353] upgrade: fix migration of old dns forward zones

2015-11-20 Thread Petr Spacek
On 20.11.2015 14:43, Martin Basti wrote: > > > On 20.11.2015 14:34, Martin Basti wrote: >> https://fedorahosted.org/freeipa/ticket/5472 >> >> Patch attached. >> >> > I forgot to attached rebased patch for ipa-4-2 ACK, thank you! -- Petr^2 Spacek -- Manage your subscription for the

Re: [Freeipa-devel] [PATCHES 509-514] replica promotion: use host credentials when setting up replication

2015-11-20 Thread Jan Cholasta
On 19.11.2015 17:43, Simo Sorce wrote: On Thu, 2015-11-19 at 15:43 +0100, Jan Cholasta wrote: Hi, the attached patches fix and . I worked around the issue of checking if the user is privileged to

Re: [Freeipa-devel] [PATCH 0353] upgrade: fix migration of old dns forward zones

2015-11-20 Thread Martin Basti
On 20.11.2015 16:43, Petr Spacek wrote: On 20.11.2015 14:43, Martin Basti wrote: On 20.11.2015 14:34, Martin Basti wrote: https://fedorahosted.org/freeipa/ticket/5472 Patch attached. I forgot to attached rebased patch for ipa-4-2 ACK, thank you! Pushed to master:

Re: [Freeipa-devel] [PATCHES 509-514] replica promotion: use host credentials when setting up replication

2015-11-20 Thread Simo Sorce
On Fri, 2015-11-20 at 16:49 +0100, Jan Cholasta wrote: > On 19.11.2015 17:43, Simo Sorce wrote: [..] > > On the patches > > -- > > 509: > > - commit says only: "aci: add IPA servers host group 'ipaservers'" > > but it does other things like changing how CA renewal certificate acis > > are added, I

Re: [Freeipa-devel] [PATCH 0064] Check if IPA is configured before attempting a winsync migration

2015-11-20 Thread Gabe Alford
Thanks. Updated patch attached. Gabe On Fri, Nov 20, 2015 at 10:36 AM, Martin Babinsky wrote: > On 11/20/2015 04:02 PM, Gabe Alford wrote: > >> Hello, >> >> Fix for https://fedorahosted.org/freeipa/ticket/5470 >> >> Thanks, >> >> Gabe >> >> >> Hi Gabe, > > patch looks

Re: [Freeipa-devel] [PATCH 0064] Check if IPA is configured before attempting a winsync migration

2015-11-20 Thread Martin Babinsky
On 11/20/2015 04:02 PM, Gabe Alford wrote: Hello, Fix for https://fedorahosted.org/freeipa/ticket/5470 Thanks, Gabe Hi Gabe, patch looks good. IMHO it would be better if you moved the check before API initialization like so: """ @@ -340,6 +340,12 @@ class

Re: [Freeipa-devel] [PATCH 0102] update idrange tests to reflect disabled modification of local ID ranges

2015-11-20 Thread Milan Kubík
On 11/20/2015 04:06 PM, Martin Babinsky wrote: When I fixed https://fedorahosted.org/freeipa/ticket/4826 I forgot to fix the corresponding xmlrpc tests. This oversight bit me today when I ran in-tree tests on my VM. Here is the patch that makes idrange tests green and shiny again. Tests are

Re: [Freeipa-devel] [patch 0025] Separated Tracker implementations into standalone package

2015-11-20 Thread Aleš Mareček
Looks good. ACK. - Original Message - > From: "Milan Kubík" > To: "freeipa-devel" > Cc: "Filip Skola" , "Ales Marecek" > Sent: Friday, November 20, 2015 3:44:30 PM > Subject: [patch 0025] Separated

[Freeipa-devel] [PATCH 515] client install: do not corrupt OpenSSH config with Match sections

2015-11-20 Thread Jan Cholasta
Hi, the attached patch fixes . Honza -- Jan Cholasta From 7e03d1370ce82843280c82c6ed13a77214ff92dd Mon Sep 17 00:00:00 2001 From: Jan Cholasta Date: Fri, 20 Nov 2015 09:35:43 +0100 Subject: [PATCH] client install: do not

[Freeipa-devel] [PATCH 0100] replica promotion: modify default.conf even if DS configuration fails

2015-11-20 Thread Martin Babinsky
Fixes https://fedorahosted.org/freeipa/ticket/5417 -- Martin^3 Babinsky -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Re: [Freeipa-devel] [PATCH 0350] raise time limit for ldapsearch in upgrade

2015-11-20 Thread Jan Cholasta
On 19.11.2015 14:13, Jan Cholasta wrote: On 19.11.2015 14:09, Martin Babinsky wrote: On 11/19/2015 01:08 PM, Martin Basti wrote: On 18.11.2015 14:26, Martin Basti wrote: On 18.11.2015 14:24, Martin Kosek wrote: On 11/18/2015 02:18 PM, Martin Basti wrote: On 18.11.2015 13:55, Martin