kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From a...@apache.org
Subject [2/2] kudu git commit: webserver-stress-itest: increase timeouts
Date Thu, 15 Sep 2016 23:53:14 GMT
webserver-stress-itest: increase timeouts

The flaky test dashboard shows a lot of ASAN and TSAN failures while waiting
for tablets to start running, so let's drastically increase that timeout.

Additionally, some ASAN builds timed out while creating the test table, so
let's also reduce the number of checkers run in ASAN.

Change-Id: I7fb042be33af0b4ecbdc39af471eab3e1c872d53
Reviewed-on: http://gerrit.cloudera.org:8080/4427
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <aserbin@cloudera.com>


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

Branch: refs/heads/master
Commit: af7f9c805c96b6ce6854105b369b6d02fbe7105a
Parents: b0b273e
Author: Adar Dembo <adar@cloudera.com>
Authored: Thu Sep 15 13:44:58 2016 -0700
Committer: Adar Dembo <adar@cloudera.com>
Committed: Thu Sep 15 23:51:18 2016 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/webserver-stress-itest.cc | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/af7f9c80/src/kudu/integration-tests/webserver-stress-itest.cc
----------------------------------------------------------------------
diff --git a/src/kudu/integration-tests/webserver-stress-itest.cc b/src/kudu/integration-tests/webserver-stress-itest.cc
index 903e3bb..d108993 100644
--- a/src/kudu/integration-tests/webserver-stress-itest.cc
+++ b/src/kudu/integration-tests/webserver-stress-itest.cc
@@ -36,8 +36,8 @@ using std::vector;
 
 // Tests that pounding the web UI doesn't cause any crashes.
 TEST_F(KuduTest, TestWebUIDoesNotCrashCluster) {
-#ifdef THREAD_SANITIZER
-  // When using TSAN, more than one checker is too much load on the cluster.
+#if defined(THREAD_SANITIZER) || defined(ADDRESS_SANITIZER)
+  // When using a sanitizer, checkers place a lot of load on the cluster.
   const int kWebUICheckers = 1;
 #else
   const int kWebUICheckers = 10;
@@ -78,7 +78,7 @@ TEST_F(KuduTest, TestWebUIDoesNotCrashCluster) {
   ASSERT_OK(cluster.Restart());
   ASSERT_OK(cluster.WaitForTabletsRunning(cluster.tablet_server(0),
                                           kNumTablets,
-                                          MonoDelta::FromSeconds(30)));
+                                          MonoDelta::FromSeconds(180)));
   NO_FATALS(cluster.AssertNoCrashes());
 }
 


Mime
View raw message