Re: [Freeipa-devel] [PATCH 0020][Tests] Added --realm parameter to master installation procedure in integration tests

2015-12-21 Thread Martin Basti


On 18.12.2015 09:37, Oleg Fayans wrote:

Sorry, my bad :)


On 12/17/2015 04:44 PM, Martin Babinsky wrote:

On 12/17/2015 04:28 PM, Oleg Fayans wrote:




Oleg,

you sent a vim swap file instead of the patch.





ACK

Pushed to:
master: b12ba14e3d07b5f2d8dd50245407da029ae7dc54
ipa-4-3: c47a5df724779183b9ab9918d49051743656f3b2

-- 
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 0020][Tests] Added --realm parameter to master installation procedure in integration tests

2015-12-18 Thread Oleg Fayans
Sorry, my bad :)


On 12/17/2015 04:44 PM, Martin Babinsky wrote:
> On 12/17/2015 04:28 PM, Oleg Fayans wrote:
>>
>>
>>
> Oleg,
> 
> you sent a vim swap file instead of the patch.
> 

-- 
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.
From 02545831dc70ec55fc0ad8c13537ea3688abb2c6 Mon Sep 17 00:00:00 2001
From: Oleg Fayans 
Date: Thu, 17 Dec 2015 16:25:46 +0100
Subject: [PATCH] Added domain realm as a parameter to master installation in
 integration tests

Without realm provided explicitly, installation calculates it automatically
from the current hostname which may be inconsistent with the configured domain
name. Which, in turn, causes failures in integration tests in the lab.
---
 ipatests/test_integration/tasks.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/ipatests/test_integration/tasks.py b/ipatests/test_integration/tasks.py
index 478a331ffee5d04e61a5d42c5fabe05427484b83..3edd9cb7367624e5f95c44a544d64740e1baef21 100644
--- a/ipatests/test_integration/tasks.py
+++ b/ipatests/test_integration/tasks.py
@@ -268,7 +268,8 @@ def install_master(host, setup_dns=True, setup_kra=False):
 
 args = [
 'ipa-server-install', '-U',
-'-r', host.domain.name,
+'-n', host.domain.name,
+'-r', host.domain.realm,
 '-p', host.config.dirman_password,
 '-a', host.config.admin_password,
 "--domain-level=%i" % host.config.domain_level,
-- 
2.4.3

-- 
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 0020][Tests] Added --realm parameter to master installation procedure in integration tests

2015-12-17 Thread Martin Babinsky

On 12/17/2015 04:28 PM, Oleg Fayans wrote:





Oleg,

you sent a vim swap file instead of the patch.

--
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


[Freeipa-devel] [PATCH 0020][Tests] Added --realm parameter to master installation procedure in integration tests

2015-12-17 Thread Oleg Fayans

-- 
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.


.freeipa-ofayans-0020-Added-domain-realm-as-a-parameter-to-master-installation.patch.swp
Description: Binary data
-- 
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