(kudu) branch master updated: KUDU-3591 Fix the flaky test FsManagerTestBase.TestAddRemoveDataDirsFuzz

[alexey]

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

alexey 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 4116941c0 KUDU-3591 Fix the flaky test 
FsManagerTestBase.TestAddRemoveDataDirsFuzz
4116941c0 is described below

commit 4116941c082be5d6e3addfe3f6b6d2ea468416e4
Author: Yingchun Lai <laiyingc...@apache.org>
AuthorDate: Sat Jul 20 23:09:24 2024 +0800

    KUDU-3591 Fix the flaky test FsManagerTestBase.TestAddRemoveDataDirsFuzz
    
    Reduce the failure rate of the test by lowering its loop
    times. For example, when FLAGS_block_manager == "logr",
    opens a data directory will open a RocksDB instance, it
    consumes more time than that if FLAGS_block_manager == "log".
    
    Change-Id: I56a03d64c8ababd63fe31fa1e7fce1efac3875e8
    Reviewed-on: http://gerrit.cloudera.org:8080/21604
    Tested-by: Kudu Jenkins
    Reviewed-by: Alexey Serbin <ale...@apache.org>
---
 src/kudu/fs/fs_manager-test.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/kudu/fs/fs_manager-test.cc b/src/kudu/fs/fs_manager-test.cc
index b030d1e03..75e26d917 100644
--- a/src/kudu/fs/fs_manager-test.cc
+++ b/src/kudu/fs/fs_manager-test.cc
@@ -1255,12 +1255,12 @@ TEST_P(FsManagerTestBase, TestAddRemoveDataDirsFuzz) {
 
 #if defined(THREAD_SANITIZER) || defined(ADDRESS_SANITIZER)
   // When using a sanitizer, reduce the loop times to get a more stable result.
-  const int kNumAttempts = 50;
+  const int kNumAttempts = 10;
 #else
   // In some situations, the tests would last too long time, so we reduce the 
loop times if not
   // AllowSlowTests(). For example, when FLAGS_block_manager == "logr", opens 
a data directory will
   // open a RocksDB instance, it consumes more time than that if 
FLAGS_block_manager == "log".
-  const int kNumAttempts = AllowSlowTests() ? 1000 : 50;
+  const int kNumAttempts = AllowSlowTests() ? 1000 : 10;
 #endif
 
   Random rng_(SeedRandom());