Hi Jan,

On 10/08/2015 10:44 AM, Jan Pazdziora wrote:
On Wed, Oct 07, 2015 at 04:13:25PM +0200, Oleg Fayans wrote:

Oleg Fayans
Quality Engineer
FreeIPA team

 From 7ab1afe5e9a8f6b28be2d5b92423eccec61248a0 Mon Sep 17 00:00:00 2001
From: Oleg Fayans <ofay...@redhat.com>
Date: Wed, 7 Oct 2015 16:08:30 +0200
Subject: [PATCH] Added a workaround for ticket N 5348

After creating signed root zone, the server requires named.service restart for 
requests to this zone to start displaying the key.
  ipatests/test_integration/test_dnssec.py | 12 +++++++++---
  1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/ipatests/test_integration/test_dnssec.py 
--- a/ipatests/test_integration/test_dnssec.py
+++ b/ipatests/test_integration/test_dnssec.py
@@ -280,7 +280,10 @@ class TestInstallDNSSECFirst(IntegrationTest):
              "--ns-rec=" + self.master.hostname
+        # A workaround for ticket N 5348
+        time.sleep(20)
+        self.master.run_command(["systemctl", "restart", 

When the ticket is addressed and these workarounds are no longer
needed -- what is our process for finding these workarounds and
reverting them, so that the tests test the real, expected behaviour?

As per discussion with Martin Basti, it was decided that this workaround will only be applied to the current 4-2 branch, not to the upstream. In upstream the issue itself will (supposedly) be solved

Also, instead of blind sleeps, wouldn't it be better to have some
polling for status of the services we are waiting for?

Since we still do not know what exactly causes the issue, it is really hard to figure out what is it that we should be polling. Otherwise I am really anti-blind-sleeps myself.

Oleg Fayans
Quality Engineer
FreeIPA team

Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to