File tree Expand file tree Collapse file tree 1 file changed +15
-3
lines changed
Expand file tree Collapse file tree 1 file changed +15
-3
lines changed Original file line number Diff line number Diff line change @@ -93,8 +93,8 @@ def parse_args():
9393 return parser .parse_args ()
9494
9595
96- FLAKY_CHECK_TEST_REPEAT_COUNT = 3
97- FLAKY_CHECK_MODULE_REPEAT_COUNT = 2
96+ FLAKY_CHECK_TEST_REPEAT_COUNT = 5
97+ FLAKY_CHECK_MODULE_REPEAT_COUNT = 100
9898
9999
100100def get_parallel_sequential_tests_to_run (
@@ -439,7 +439,19 @@ def main():
439439 for failed_suit in failed_suits :
440440 failed_tests_files .append (f"tests/integration/{ failed_suit } " )
441441
442- if failed_suits :
442+ # For flaky checks, always collect logs to allow inspection of server logs
443+ # even when tests pass
444+ if is_flaky_check :
445+ all_test_suits = set ()
446+ for test_module in parallel_test_modules + sequential_test_modules :
447+ suit = test_module .split ("/" )[0 ]
448+ all_test_suits .add (suit )
449+ for suit in all_test_suits :
450+ suit_path = f"tests/integration/{ suit } "
451+ if suit_path not in failed_tests_files :
452+ failed_tests_files .append (suit_path )
453+
454+ if failed_suits or is_flaky_check :
443455 attached_files .append (
444456 Utils .compress_files_gz (failed_tests_files , f"{ temp_path } /logs.tar.gz" )
445457 )
You can’t perform that action at this time.
0 commit comments