[kudu] branch master updated: webserver-crawl-itest: reduce test load

[adar]

This is an automated email from the ASF dual-hosted git repository.

adar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 618b711  webserver-crawl-itest: reduce test load
618b711 is described below

commit 618b711ca0f42f2b891dcdd9a2d1583a95bbe1b5
Author: Adar Dembo <a...@cloudera.com>
AuthorDate: Mon Dec 2 10:32:25 2019 -0800

    webserver-crawl-itest: reduce test load
    
    The flaky test dashboard shows a few tests failing with:
    
      /.../src/kudu/integration-tests/webserver-crawl-itest.cc:299: Failure
      Failed
      Bad status: Network error: curl error: SSL connect error
    
    This appears to be happening in ASAN and TSAN tests. I'm not sure why
    anything is timing out as there's no curl timeout configured. In any case,
    I've added a SCOPED_TRACE so we can see whether it's always the same URL
    that's failing, and I've also removed the read threads from the TestWorkload
    as they don't serve any purpose in the test (apart from generating load).
    
    Change-Id: I0b59e9bcf418cde51cb531d7c0d5b38d5f13cd47
    Reviewed-on: http://gerrit.cloudera.org:8080/14815
    Reviewed-by: Alexey Serbin <aser...@cloudera.com>
    Tested-by: Kudu Jenkins
---
 src/kudu/integration-tests/webserver-crawl-itest.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/kudu/integration-tests/webserver-crawl-itest.cc 
b/src/kudu/integration-tests/webserver-crawl-itest.cc
index ac85e80..2ed35e7 100644
--- a/src/kudu/integration-tests/webserver-crawl-itest.cc
+++ b/src/kudu/integration-tests/webserver-crawl-itest.cc
@@ -168,10 +168,9 @@ TEST_P(WebserverCrawlITest, TestAllWebPages) {
   ExternalMiniCluster cluster(std::move(opts));
   ASSERT_OK(cluster.Start());
 
-  // Create a table and write soem data so that log anchors page gets 
populated.
+  // Create a table and write some data so that the log anchor pages get 
populated.
   TestWorkload work(&cluster);
   work.set_num_replicas(3);
-  work.set_num_read_threads(4);
   work.set_num_tablets(6);
   work.Setup();
   work.Start();
@@ -296,6 +295,7 @@ TEST_P(WebserverCrawlITest, TestAllWebPages) {
     string host = ret == string::npos ? url : url.substr(0, ret);
 
     // Every link should be reachable.
+    SCOPED_TRACE(url);
     ASSERT_OK(curl.FetchURL(url, &response, headers));
     string resp_str = response.ToString();
     SCOPED_TRACE(resp_str);