Re: [Freeipa-devel] [PATCH 0328] DNSSEC CI: Wait until DS records are replicated

2015-10-22 Thread Petr Spacek
On 16.10.2015 16:51, Martin Basti wrote:
> Drill command sometimes fail on replica due to long replication time, this
> patch adding check that waits until record is replicated.
> 
> Patch attached.

ACK

-- 
Petr^2 Spacek

-- 
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 0328] DNSSEC CI: Wait until DS records are replicated

2015-10-22 Thread Martin Basti



On 22.10.2015 17:36, Petr Spacek wrote:

On 16.10.2015 16:51, Martin Basti wrote:

Drill command sometimes fail on replica due to long replication time, this
patch adding check that waits until record is replicated.

Patch attached.

ACK


Pushed to:
master: f2032ca2cab3048021956707ec4ff74be69108d4
ipa-4-2: edaf4665e04df2614b59e4319a62030566d1b52d

--
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 0328] DNSSEC CI: Wait until DS records are replicated

2015-10-16 Thread Martin Basti
Drill command sometimes fail on replica due to long replication time, 
this patch adding check that waits until record is replicated.


Patch attached.
From bce95f9d16fd3a920ca50952be75022d89682699 Mon Sep 17 00:00:00 2001
From: Martin Basti 
Date: Fri, 16 Oct 2015 14:09:37 +0200
Subject: [PATCH] DNSSEC CI: wait until DS records is replicated

In some cases replication may take much more time than we expected. This
patch adds explicit cech if DS records has been replicated.
---
 ipatests/test_integration/test_dnssec.py | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/ipatests/test_integration/test_dnssec.py b/ipatests/test_integration/test_dnssec.py
index 66e67a6efbe1db767f8b7102d2928be775e723af..5d6acb7cc0d843929fd23e5a07f37803cbfdb792 100644
--- a/ipatests/test_integration/test_dnssec.py
+++ b/ipatests/test_integration/test_dnssec.py
@@ -350,6 +350,12 @@ class TestInstallDNSSECFirst(IntegrationTest):
 
 self.master.run_command(args)
 
+# wait until DS records it replicated
+assert wait_until_record_is_signed(
+self.replicas[0].ip, example_test_zone, self.log, timeout=100,
+rtype="DS"
+), "No DS record of '%s' returned from replica" % example_test_zone
+
 # extract DSKEY from root zone
 ans = resolve_with_dnssec(self.master.ip, root_zone, self.log,
   rtype="DNSKEY")
-- 
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