Skip to content

Conversation

@boomanaiden154
Copy link
Contributor

To enable disambiguating the platform later when we look at identifying tests failing at HEAD/flaky tests.

To enable disambiguating the platform later when we look at identifying
tests failing at HEAD/flaky tests.
@boomanaiden154 boomanaiden154 requested a review from cmtice October 16, 2025 16:17
@llvmbot llvmbot added the infrastructure Bugs about LLVM infrastructure label Oct 16, 2025
@llvmbot
Copy link
Member

llvmbot commented Oct 16, 2025

@llvm/pr-subscribers-infrastructure

Author: Aiden Grossman (boomanaiden154)

Changes

To enable disambiguating the platform later when we look at identifying tests failing at HEAD/flaky tests.


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

1 Files Affected:

  • (modified) .ci/premerge_advisor_upload.py (+2)
diff --git a/.ci/premerge_advisor_upload.py b/.ci/premerge_advisor_upload.py
index 84214f8a6d7ef..398fe1fa8ce96 100644
--- a/.ci/premerge_advisor_upload.py
+++ b/.ci/premerge_advisor_upload.py
@@ -23,11 +23,13 @@ def main(commit_sha, workflow_run_number, build_log_files):
     )
     test_failures = generate_test_report_lib.get_failures(junit_objects)
     source = "pull_request" if "GITHUB_ACTIONS" in os.environ else "postcommit"
+    current_platform = f"{platform.system()}-{platform.machine()}".lower()
     failure_info = {
         "source_type": source,
         "base_commit_sha": commit_sha,
         "source_id": workflow_run_number,
         "failures": [],
+        "platform": current_platform
     }
     if test_failures:
         for name, failure_message in test_failures:

@github-actions
Copy link

github-actions bot commented Oct 16, 2025

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

@boomanaiden154 boomanaiden154 merged commit d58b5a6 into llvm:main Oct 16, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infrastructure Bugs about LLVM infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants