From 95864d7ca3f177886b9c8068283dd29855695e69 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nathan=20Gau=C3=ABr?= Date: Fri, 17 Oct 2025 13:25:07 +0200 Subject: [PATCH] [CI] Fix unpacking in premerge advisor upload Seems like the function returns a dict, not a list of tuples like the other. Not sure of the schema or the design for this, so fixing the code without using the test-suite name. But this might have to be addressed once the owner can take a look. --- .ci/premerge_advisor_upload.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/.ci/premerge_advisor_upload.py b/.ci/premerge_advisor_upload.py index dda4ad24d7ac4..1fc2423dd24fc 100644 --- a/.ci/premerge_advisor_upload.py +++ b/.ci/premerge_advisor_upload.py @@ -32,8 +32,11 @@ def main(commit_sha, workflow_run_number, build_log_files): "platform": current_platform, } if test_failures: - for name, failure_message in test_failures: - failure_info["failures"].append({"name": name, "message": failure_message}) + for _, failures in test_failures.items(): + for name, failure_message in failures: + failure_info["failures"].append( + {"name": name, "message": failure_message} + ) else: ninja_failures = generate_test_report_lib.find_failure_in_ninja_logs(ninja_logs) for name, failure_message in ninja_failures: