kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ale...@apache.org
Subject kudu git commit: [security-faults-itest] fix test flakiness
Date Fri, 07 Apr 2017 18:50:50 GMT
Repository: kudu
Updated Branches:
  refs/heads/master 9badc5003 -> b95d3a4fa


[security-faults-itest] fix test flakiness

When running the security-faults-itest on a slow VM with other
activities in parallel, it might take up to 8 seconds to start a master.
So, bumping the ticket TTL to higher value so Kudu cluster with 3
masters finishes starting up when initially acquired tickets are valid.

Also, updated Raft parameters interval to speed up leader election.

Change-Id: I132426ee37358a228cf3f5d1fef9e9ec08610b16
Reviewed-on: http://gerrit.cloudera.org:8080/6586
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <adar@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/b95d3a4f
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/b95d3a4f
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/b95d3a4f

Branch: refs/heads/master
Commit: b95d3a4facec6785fe00d0763ae79b0eed462991
Parents: 9badc50
Author: Alexey Serbin <aserbin@cloudera.com>
Authored: Thu Apr 6 19:18:45 2017 -0700
Committer: Alexey Serbin <aserbin@cloudera.com>
Committed: Fri Apr 7 18:45:13 2017 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/security-faults-itest.cc | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/b95d3a4f/src/kudu/integration-tests/security-faults-itest.cc
----------------------------------------------------------------------
diff --git a/src/kudu/integration-tests/security-faults-itest.cc b/src/kudu/integration-tests/security-faults-itest.cc
index 6651e85..f71c4d1 100644
--- a/src/kudu/integration-tests/security-faults-itest.cc
+++ b/src/kudu/integration-tests/security-faults-itest.cc
@@ -52,11 +52,7 @@ namespace kudu {
 class SecurityComponentsFaultsITest : public KuduTest {
  public:
   SecurityComponentsFaultsITest()
-#if defined(ADDRESS_SANITIZER) || defined(THREAD_SANITIZER)
       : krb_lifetime_seconds_(64),
-#else
-      : krb_lifetime_seconds_(16),
-#endif
         num_masters_(3),
         num_tservers_(3) {
 
@@ -77,11 +73,22 @@ class SecurityComponentsFaultsITest : public KuduTest {
     const vector<string> common_flags = {
       // Enable tracing of successful KRPC negotiations as well.
       "--rpc_trace_negotiation",
+
+      // Speed up Raft elections.
+      "--raft_heartbeat_interval_ms=25",
+      "--leader_failure_exp_backoff_max_delta_ms=1000",
     };
     std::copy(common_flags.begin(), common_flags.end(),
         std::back_inserter(cluster_opts_.extra_master_flags));
     std::copy(common_flags.begin(), common_flags.end(),
         std::back_inserter(cluster_opts_.extra_tserver_flags));
+
+    const vector<string> tserver_flags = {
+      // Decreasing TS->master heartbeat interval speeds up the test.
+      "--heartbeat_interval_ms=25",
+    };
+    std::copy(tserver_flags.begin(), tserver_flags.end(),
+        std::back_inserter(cluster_opts_.extra_tserver_flags));
   }
 
   Status StartCluster() {


Mime
View raw message