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

2015-12-09 Thread Jan Cholasta
On 9.12.2015 16:39, Jan Cholasta wrote: On 7.12.2015 08:14, Jan Cholasta wrote: On 6.12.2015 21:32, Martin Basti wrote: On 04.12.2015 16:58, Simo Sorce wrote: On Fri, 2015-12-04 at 15:39 +0100, Jan Cholasta wrote: On 4.12.2015 15:16, Jan Cholasta wrote: On 4.12.2015 15:12, Jan Cholasta wro

Re: [Freeipa-devel] [PATCH 0071] replica: Fix ipa-replica-install with replica file (domain, level 0).

2015-12-09 Thread Jan Cholasta
On 9.12.2015 08:31, David Kupka wrote: On 08/12/15 16:33, Tomas Babej wrote: On 12/08/2015 04:20 PM, Oleg Fayans wrote: ACK. The initial issue is fixed. On 12/08/2015 03:03 PM, David Kupka wrote: https://fedorahosted.org/freeipa/ticket/5531 Can we get some more love for the patch and

Re: [Freeipa-devel] [PATCH 560] Allow to set allowed krb authz data type per user

2015-12-09 Thread Simo Sorce
Sent the wrong patch, attached the one that actually compiles. - Original Message - > From: "Simo Sorce" > To: "Alexander Bokovoy" > Cc: "Simo Sorce" , "Jan Cholasta" , > "freeipa-devel" > Sent: Wednesday, December 9, 2015 2:18:23 PM > Subject: Re: [Freeipa-devel] [PATCH 560] Allow to

Re: [Freeipa-devel] [PATCHES 516-517] spec file: put Python modules into standalone packages

2015-12-09 Thread Petr Vobornik
On 12/07/2015 04:21 PM, Jan Cholasta wrote: Hi, the attached patches partially fix . This is done to allow the addition of Python 3 packages, see . See commit messages for mor

Re: [Freeipa-devel] [PATCH 560] Allow to set allowed krb authz data type per user

2015-12-09 Thread Simo Sorce
- Original Message - > From: "Alexander Bokovoy" > To: "Simo Sorce" > Cc: "Jan Cholasta" , "freeipa-devel" > > Sent: Tuesday, December 1, 2015 3:07:32 AM > Subject: Re: [Freeipa-devel] [PATCH 560] Allow to set allowed krb authz data > type per user > > On Wed, 25 Nov 2015, Simo Sorce

[Freeipa-devel] [PATCH 0373] Upgrade: Fix IPA version comparison

2015-12-09 Thread Martin Basti
https://fedorahosted.org/freeipa/ticket/5535 Patch attached. From 8ef93485d61e8732166fb0c5b6c4559209740f3e Mon Sep 17 00:00:00 2001 From: Martin Basti Date: Wed, 9 Dec 2015 18:53:35 +0100 Subject: [PATCH] Fix version comparison Use RPM library to compare vendor versions of IPA for redhat platfo

Re: [Freeipa-devel] [PATCH 0394] topology: Make sure the old 'realm' topology suffix is not

2015-12-09 Thread Martin Basti
On 09.12.2015 09:43, Martin Basti wrote: On 08.12.2015 17:32, Martin Babinsky wrote: On 12/08/2015 04:53 PM, Tomas Babej wrote: On 12/08/2015 02:28 PM, Tomas Babej wrote: Hi, The old 'realm' topology suffix is no longer used, however, it was being created on masters with version 4.2.3

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

2015-12-09 Thread Jan Cholasta
On 7.12.2015 08:14, Jan Cholasta wrote: On 6.12.2015 21:32, Martin Basti wrote: On 04.12.2015 16:58, Simo Sorce wrote: On Fri, 2015-12-04 at 15:39 +0100, Jan Cholasta wrote: On 4.12.2015 15:16, Jan Cholasta wrote: On 4.12.2015 15:12, Jan Cholasta wrote: On 4.12.2015 11:15, Petr Vobornik wr

Re: [Freeipa-devel] [PATCHES 523-525] replica install: add remote connection check over API

2015-12-09 Thread Jan Cholasta
On 9.12.2015 14:52, Jan Cholasta wrote: On 9.12.2015 10:02, Jan Cholasta wrote: Hi, the attached patches fix . Note that this needs selinux-policy fix to work, so put SELinux into permissive mode for testing:

Re: [Freeipa-devel] [PATCH 0098-0099] domain level 1 topology checks during IPA server uninstall

2015-12-09 Thread Martin Babinsky
On 12/09/2015 03:48 PM, Jan Cholasta wrote: On 2.12.2015 14:19, Martin Basti wrote: On 02.12.2015 14:10, Martin Basti wrote: On 02.12.2015 14:08, Martin Babinsky wrote: On 12/02/2015 10:45 AM, Martin Babinsky wrote: On 12/01/2015 02:40 PM, Martin Babinsky wrote: On 11/30/2015 08:34 PM, M

Re: [Freeipa-devel] [PATCH 0098-0099] domain level 1 topology checks during IPA server uninstall

2015-12-09 Thread Jan Cholasta
On 2.12.2015 14:19, Martin Basti wrote: On 02.12.2015 14:10, Martin Basti wrote: On 02.12.2015 14:08, Martin Babinsky wrote: On 12/02/2015 10:45 AM, Martin Babinsky wrote: On 12/01/2015 02:40 PM, Martin Babinsky wrote: On 11/30/2015 08:34 PM, Martin Basti wrote: On 30.11.2015 18:41, Ma

Re: [Freeipa-devel] [PATCHES 523-525] replica install: add remote connection check over API

2015-12-09 Thread Jan Cholasta
On 9.12.2015 10:02, Jan Cholasta wrote: Hi, the attached patches fix . Note that this needs selinux-policy fix to work, so put SELinux into permissive mode for testing: . -- Jan Cholasta --

Re: [Freeipa-devel] [PATCH 0065] ipa-replica-install prints incorrect error message when replica is already installed

2015-12-09 Thread Gabe Alford
Fixed. Updated patch attached. On Wed, Dec 9, 2015 at 2:37 AM, Martin Basti wrote: > NACK > > Patch contains syntax error, missing brace > > ipaserver/install/server/replicainstall.py:850: [E0001(syntax-error), ] > invalid syntax) > > Martin > > > On 09.12.2015 07:08, Jan Cholasta wrote: > >> LG

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

2015-12-09 Thread Martin Babinsky
On 12/09/2015 11:29 AM, Lenka Doudova wrote: On 12/09/2015 10:13 AM, Martin Basti wrote: On 09.12.2015 09:41, Lenka Doudova wrote: Hi, attaching fixed patches for master and ipa-4-2 branch. Also fixes test accordingly to https://fedorahosted.org/freeipa/ticket/5387. Lenka On 11/20/2015 1

Re: [Freeipa-devel] [PATCH] 0748 Handle encoding for ipautil.run

2015-12-09 Thread Petr Viktorin
On 12/04/2015 12:49 PM, Jan Cholasta wrote: > On 4.12.2015 12:43, Petr Viktorin wrote: >> On 12/04/2015 12:15 PM, Jan Cholasta wrote: >>> On 4.12.2015 10:53, Petr Viktorin wrote: On 12/04/2015 08:51 AM, Jan Cholasta wrote: > On 3.12.2015 15:42, Petr Viktorin wrote: >> On 12/03/2015 10:

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

2015-12-09 Thread Lenka Doudova
On 12/09/2015 10:13 AM, Martin Basti wrote: On 09.12.2015 09:41, Lenka Doudova wrote: Hi, attaching fixed patches for master and ipa-4-2 branch. Also fixes test accordingly to https://fedorahosted.org/freeipa/ticket/5387. Lenka On 11/20/2015 12:13 PM, Martin Babinsky wrote: On 11/19/20

Re: [Freeipa-devel] [PATCH] First part of the replica promotion tests + testplan

2015-12-09 Thread Oleg Fayans
Hi Martin On 12/09/2015 10:30 AM, Martin Basti wrote: > > > On 08.12.2015 23:48, Oleg Fayans wrote: >> Substituted a hardcoded suffix name with a constant DOMAIN_SUFFIX_NAME >> >> On 12/08/2015 02:33 PM, Oleg Fayans wrote: >>> Hi all, >>> >>> >>> The patches are rebased against the current maste

Re: [Freeipa-devel] [PATCH 0065] ipa-replica-install prints incorrect error message when replica is already installed

2015-12-09 Thread Martin Basti
NACK Patch contains syntax error, missing brace ipaserver/install/server/replicainstall.py:850: [E0001(syntax-error), ] invalid syntax) Martin On 09.12.2015 07:08, Jan Cholasta wrote: LGTM On 8.12.2015 17:04, Gabe Alford wrote: Updated patch attached. On Tue, Dec 8, 2015 at 8:27 AM, Mart

Re: [Freeipa-devel] [PATCHES 523-525] replica install: add remote connection check over API

2015-12-09 Thread Jan Cholasta
On 9.12.2015 10:34, Alexander Bokovoy wrote: On Wed, 09 Dec 2015, Jan Cholasta wrote: From 142fd5364cf9d31d7e2c35959516ac8d9054c9da Mon Sep 17 00:00:00 2001 From: Jan Cholasta Date: Wed, 9 Dec 2015 08:17:07 +0100 Subject: [PATCH 1/3] build: put oddjob scripts into separate directory https://fe

Re: [Freeipa-devel] [PATCHES 523-525] replica install: add remote connection check over API

2015-12-09 Thread Alexander Bokovoy
On Wed, 09 Dec 2015, Jan Cholasta wrote: From 142fd5364cf9d31d7e2c35959516ac8d9054c9da Mon Sep 17 00:00:00 2001 From: Jan Cholasta Date: Wed, 9 Dec 2015 08:17:07 +0100 Subject: [PATCH 1/3] build: put oddjob scripts into separate directory https://fedorahosted.org/freeipa/ticket/5497 --- freeipa

Re: [Freeipa-devel] [PATCH] First part of the replica promotion tests + testplan

2015-12-09 Thread Martin Basti
On 08.12.2015 23:48, Oleg Fayans wrote: Substituted a hardcoded suffix name with a constant DOMAIN_SUFFIX_NAME On 12/08/2015 02:33 PM, Oleg Fayans wrote: Hi all, The patches are rebased against the current master. On 12/02/2015 05:10 PM, Martin Basti wrote: On 02.12.2015 16:18, Oleg Faya

Re: [Freeipa-devel] [PATCH 522] replica promotion: allow OTP bulk client enrollment

2015-12-09 Thread Martin Basti
On 08.12.2015 13:19, Martin Basti wrote: On 08.12.2015 13:09, Jan Cholasta wrote: On 8.12.2015 12:49, Martin Basti wrote: On 08.12.2015 10:31, Martin Basti wrote: On 08.12.2015 08:52, Jan Cholasta wrote: On 7.12.2015 21:11, Martin Basti wrote: On 07.12.2015 08:21, Jan Cholasta wrot

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

2015-12-09 Thread Martin Basti
On 09.12.2015 09:41, Lenka Doudova wrote: Hi, attaching fixed patches for master and ipa-4-2 branch. Also fixes test accordingly to https://fedorahosted.org/freeipa/ticket/5387. Lenka On 11/20/2015 12:13 PM, Martin Babinsky wrote: On 11/19/2015 10:34 AM, Petr Viktorin wrote: On 11/19/201

[Freeipa-devel] [PATCHES 523-525] replica install: add remote connection check over API

2015-12-09 Thread Jan Cholasta
Hi, the attached patches fix . Honza -- Jan Cholasta From 142fd5364cf9d31d7e2c35959516ac8d9054c9da Mon Sep 17 00:00:00 2001 From: Jan Cholasta Date: Wed, 9 Dec 2015 08:17:07 +0100 Subject: [PATCH 1/3] build: put oddjob scripts into separate directo

Re: [Freeipa-devel] [PATCH 0394] topology: Make sure the old 'realm' topology suffix is not

2015-12-09 Thread Martin Basti
On 08.12.2015 17:32, Martin Babinsky wrote: On 12/08/2015 04:53 PM, Tomas Babej wrote: On 12/08/2015 02:28 PM, Tomas Babej wrote: Hi, The old 'realm' topology suffix is no longer used, however, it was being created on masters with version 4.2.3 and later. Make sure it's properly removed

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

2015-12-09 Thread Lenka Doudova
Hi, attaching fixed patches for master and ipa-4-2 branch. Also fixes test accordingly to https://fedorahosted.org/freeipa/ticket/5387. Lenka On 11/20/2015 12:13 PM, Martin Babinsky wrote: On 11/19/2015 10:34 AM, Petr Viktorin wrote: On 11/19/2015 09:30 AM, Lenka Doudova wrote: On 11/18/2015

Re: [Freeipa-devel] [PATCH 0112] CI tests: ignore disconnected domain level 1 topology on IPA master teardown

2015-12-09 Thread Martin Basti
On 09.12.2015 08:26, Oleg Fayans wrote: ACK On 12/09/2015 07:37 AM, Martin Babinsky wrote: On 12/07/2015 01:53 PM, Martin Babinsky wrote: On 12/07/2015 12:07 PM, Oleg Fayans wrote: Hi Martin, CONFIGURED_DOMAIN_LEVEL is declared, but not used. The rest looks fine to me On 12/07/2015 11:05