Skip to content

Commit 45fd898

Browse files
committed
Run many times the flaky check, collecting all logs
1 parent 5dcdae8 commit 45fd898

File tree

1 file changed

+15
-3
lines changed

1 file changed

+15
-3
lines changed

ci/jobs/integration_test_job.py

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff 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

100100
def 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
)

0 commit comments

Comments
 (0)