Skip to content

Commit e392cdc

Browse files
committed
debugging missing integration tests logs
1 parent 950e14b commit e392cdc

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

ci/jobs/scripts/integration_tests_runner.py

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -513,7 +513,7 @@ def get_log_paths(test_name):
513513
def test_is_known_fail(test_name, test_logs, debug_log_file):
514514
if test_logs is None:
515515
debug_log_file.write(
516-
f"WARNING: Test {test_name} has no logs - cannot match message patterns\n"
516+
f"WARNING: Test '{test_name}' has no logs - cannot match message patterns\n"
517517
)
518518

519519
matching_rules = []
@@ -577,9 +577,12 @@ def matches_substring(substring, log, is_regex):
577577

578578
# Should only care about the most recent log file
579579
log_path = get_log_paths(failed_test)[0]
580-
test_log = extract_fail_logs(log_path).get(
581-
failed_test.split("::")[-1]
582-
)
580+
test_logs = extract_fail_logs(log_path)
581+
test_log = test_logs.get(failed_test.split("::")[-1])
582+
if test_log is None:
583+
log_file.write(
584+
f"WARNING: Test '{failed_test}' has no logs among {test_logs.keys()}\n"
585+
)
583586
known_fail_reason = test_is_known_fail(
584587
failed_test, test_log, log_file
585588
)

0 commit comments

Comments
 (0)