You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2017/04/07 18:50:50 UTC
kudu git commit: [security-faults-itest] fix test flakiness
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 <ad...@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 <as...@cloudera.com>
Authored: Thu Apr 6 19:18:45 2017 -0700
Committer: Alexey Serbin <as...@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() {