David Ribeiro Alves has submitted this change and it was merged. Change subject: [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 <t...@apache.org> Tested-by: Kudu Jenkins --- M src/kudu/integration-tests/external_mini_cluster.cc 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6230 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1389361a3811b265178e3a174bf94af44d40fee9 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>