You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zookeeper.apache.org by an...@apache.org on 2019/01/14 14:36:37 UTC
[zookeeper] branch branch-3.5 updated: ZOOKEEPER-3202: Add timing
margin to improve reliability of testClientServerSSL()
This is an automated email from the ASF dual-hosted git repository.
andor pushed a commit to branch branch-3.5
in repository https://gitbox.apache.org/repos/asf/zookeeper.git
The following commit(s) were added to refs/heads/branch-3.5 by this push:
new 409ac0c ZOOKEEPER-3202: Add timing margin to improve reliability of testClientServerSSL()
409ac0c is described below
commit 409ac0c036ff9e6f3860a832d4fbd5319487003f
Author: Michael Edwards <Michael Edwards>
AuthorDate: Mon Jan 14 07:36:15 2019 -0700
ZOOKEEPER-3202: Add timing margin to improve reliability of testClientServerSSL()
Allowing just 5 seconds for 3 quorum peers to start and elect a leader is a bit tight, at least when running 4 test processes in parallel inside a (Linux) Docker container on a (non-Linux) laptop. Add up to 10 seconds of extra margin.
Author: Michael Edwards <Michael Edwards>
Reviewers: andor@apache.org
Closes #723 from mkedwards/ZOOKEEPER-3202
(cherry picked from commit 14eefcb7bbd1cc93d4738998ca7343cd9cae8277)
Signed-off-by: Andor Molnar <an...@apache.org>
---
.../src/test/java/org/apache/zookeeper/test/ClientSSLTest.java | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/zookeeper-server/src/test/java/org/apache/zookeeper/test/ClientSSLTest.java b/zookeeper-server/src/test/java/org/apache/zookeeper/test/ClientSSLTest.java
index a09fc89..1504834 100644
--- a/zookeeper-server/src/test/java/org/apache/zookeeper/test/ClientSSLTest.java
+++ b/zookeeper-server/src/test/java/org/apache/zookeeper/test/ClientSSLTest.java
@@ -95,6 +95,10 @@ public class ClientSSLTest extends QuorumPeerTestBase {
mt[i].start();
}
+ // Add some timing margin for the quorum to elect a leader
+ // (without this margin, timeouts have been observed in parallel test runs)
+ ClientBase.waitForServerUp("127.0.0.1:" + clientPorts[0], 2 * TIMEOUT);
+
// Servers have been set up. Now go test if secure connection is successful.
for (int i = 0; i < SERVER_COUNT; i++) {
Assert.assertTrue("waiting for server " + i + " being up",