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 <[email protected]> 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 <[email protected]> Authored: Thu Apr 6 19:18:45 2017 -0700 Committer: Alexey Serbin <[email protected]> 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() {
