[CI] Fix unpacking in premerge advisor upload

[Keenuts]

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.

[llvmbot]

@llvm/pr-subscribers-infrastructure

Author: Nathan Gauër (Keenuts)

Changes

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.

---

Full diff: https://github.com/llvm/llvm-project/pull/163944.diff

1 Files Affected:

• (modified) .ci/premerge_advisor_upload.py (+3-2)

diff --git a/.ci/premerge_advisor_upload.py b/.ci/premerge_advisor_upload.py
index dda4ad24d7ac4..fe3eba90f18d8 100644
--- a/.ci/premerge_advisor_upload.py
+++ b/.ci/premerge_advisor_upload.py
@@ -32,8 +32,9 @@ 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:

[github-actions]

✅ With the latest revision this PR passed the Python code formatter.

[Keenuts]

CI failures now seems unrelated, those are actual LLVM test failures unrelated to this script fix. Merging this now as the script failure could hide actual failures.