You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by to...@apache.org on 2017/03/03 01:25:42 UTC

[7/7] kudu git commit: [flaky tests] Allow for the master to take longer to start

[flaky tests] Allow for the master to take longer to start

With the additional work that the master has to perform for
security it's taking longer to boot, causing some tests to
timeout while waiting for the master to be ready.
Example from alter_schema_randomized-test:

I0302 03:22:13.666565 10295 catalog_manager.cc:850] Loading CA info into memory...
I0302 03:22:14.347914 10295 catalog_manager.cc:894] Loading token signing keys...
F0302 03:22:15.072451 20059 alter_table-randomized-test.cc:99] Check failed: _s.ok() Bad status: Timed out: Timed out after 30s waiting for master (127.0.0.1:46514) startup

This patch just doubles the time we allow the master to
take to boot up before we give up with a Status::TimedOut().

Change-Id: I1389361a3811b265178e3a174bf94af44d40fee9
Reviewed-on: http://gerrit.cloudera.org:8080/6230
Reviewed-by: Todd Lipcon <to...@apache.org>
Tested-by: Kudu Jenkins


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

Branch: refs/heads/master
Commit: 889a04ad9f5d4c28758c0ad1f5dbe52f43d93550
Parents: 8db8968
Author: David Alves <dr...@apache.org>
Authored: Thu Mar 2 14:23:07 2017 -0800
Committer: David Ribeiro Alves <dr...@apache.org>
Committed: Fri Mar 3 00:38:38 2017 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/external_mini_cluster.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/889a04ad/src/kudu/integration-tests/external_mini_cluster.cc
----------------------------------------------------------------------
diff --git a/src/kudu/integration-tests/external_mini_cluster.cc b/src/kudu/integration-tests/external_mini_cluster.cc
index a57f92a..a9b5bab 100644
--- a/src/kudu/integration-tests/external_mini_cluster.cc
+++ b/src/kudu/integration-tests/external_mini_cluster.cc
@@ -76,7 +76,7 @@ static const char* const kWildcardIpAddr = "0.0.0.0";
 static const char* const kLoopbackIpAddr = "127.0.0.1";
 static double kProcessStartTimeoutSeconds = 30.0;
 static double kTabletServerRegistrationTimeoutSeconds = 15.0;
-static double kMasterCatalogManagerTimeoutSeconds = 30.0;
+static double kMasterCatalogManagerTimeoutSeconds = 60.0;
 
 #if defined(__APPLE__)
 static ExternalMiniClusterOptions::BindMode kBindMode =