mirielka's pull request #74: "[master, ipa-4-4] Tests: Add krb5kdc.service 
restart to integration trust tests" was opened

PR body:
"""
krb5kdc.service restart is necessary for proper running of integration trust
related tests.

https://fedorahosted.org/freeipa/ticket/6322
"""

See the full pull-request at https://github.com/freeipa/freeipa/pull/74
... or pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/74/head:pr74
git checkout pr74
From a220977bcd55137a6f96bc60987e24978390332f Mon Sep 17 00:00:00 2001
From: Lenka Doudova <ldoud...@redhat.com>
Date: Tue, 13 Sep 2016 08:17:53 +0200
Subject: [PATCH] Tests: Add krb5kdc.service restart to integration trust tests

krb5kdc.service restart is necessary for proper running of integration trust
related tests.

https://fedorahosted.org/freeipa/ticket/6322
---
 ipatests/test_integration/tasks.py      | 2 ++
 ipatests/test_integration/test_trust.py | 3 ---
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/ipatests/test_integration/tasks.py b/ipatests/test_integration/tasks.py
index c60d436..b8defa7 100644
--- a/ipatests/test_integration/tasks.py
+++ b/ipatests/test_integration/tasks.py
@@ -490,6 +490,8 @@ def establish_trust_with_ad(master, ad_domain, extra_args=()):
                         stdin_text=master.config.ad_admin_password)
     master.run_command(['smbcontrol', 'all', 'debug', '1'])
     clear_sssd_cache(master)
+    master.run_command(['systemctl', 'restart', 'krb5kdc.service'])
+    time.sleep(60)
 
 
 def remove_trust_with_ad(master, ad_domain):
diff --git a/ipatests/test_integration/test_trust.py b/ipatests/test_integration/test_trust.py
index 69418dc..6b30338 100644
--- a/ipatests/test_integration/test_trust.py
+++ b/ipatests/test_integration/test_trust.py
@@ -24,7 +24,6 @@
 from ipatests.test_integration import tasks
 from ipatests.test_integration import util
 from ipaplatform.paths import paths
-import time
 
 
 class ADTrustBase(IntegrationTest):
@@ -400,8 +399,6 @@ def test_upn_user_resolution_in_nonposix_trust(self):
 
     def test_upn_user_authentication(self):
         """ Check that AD user with UPN can authenticate in IPA """
-        self.master.run_command(['systemctl', 'restart', 'krb5kdc'])
-        time.sleep(60)
         self.master.run_command(['kinit', '-C', '-E', self.upn_principal],
                                 stdin_text=self.upn_password)
 
-- 
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

Reply via email to