Skip to content

Conversation

@boomanaiden154
Copy link
Contributor

So that we do not have to worry about synchronizing data between the two
clusters. This also enables this script to work on AArch64, although
we'll look at enabling that later.

Created using spr 1.3.7
@boomanaiden154 boomanaiden154 requested a review from cmtice October 24, 2025 23:48
@llvmbot llvmbot added the infrastructure Bugs about LLVM infrastructure label Oct 24, 2025
@llvmbot
Copy link
Member

llvmbot commented Oct 24, 2025

@llvm/pr-subscribers-infrastructure

Author: Aiden Grossman (boomanaiden154)

Changes

So that we do not have to worry about synchronizing data between the two
clusters. This also enables this script to work on AArch64, although
we'll look at enabling that later.


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

1 Files Affected:

  • (modified) .ci/premerge_advisor_upload.py (+6-4)
diff --git a/.ci/premerge_advisor_upload.py b/.ci/premerge_advisor_upload.py
index 1fc2423dd24fc..a50c8c4765bef 100644
--- a/.ci/premerge_advisor_upload.py
+++ b/.ci/premerge_advisor_upload.py
@@ -12,9 +12,10 @@
 
 import generate_test_report_lib
 
-PREMERGE_ADVISOR_URL = (
-    "http://premerge-advisor.premerge-advisor.svc.cluster.local:5000/upload"
-)
+PREMERGE_ADVISOR_URLS = [
+    "http://34.82.126.63:5000/upload",
+    "http://136.114.125.23:5000/upload",
+]
 
 
 def main(commit_sha, workflow_run_number, build_log_files):
@@ -41,7 +42,8 @@ def main(commit_sha, workflow_run_number, build_log_files):
         ninja_failures = generate_test_report_lib.find_failure_in_ninja_logs(ninja_logs)
         for name, failure_message in ninja_failures:
             failure_info["failures"].append({"name": name, "message": failure_message})
-    requests.post(PREMERGE_ADVISOR_URL, json=failure_info)
+    for premerge_advisor_url in PREMERGE_ADVISOR_URLS:
+        requests.post(premerge_advisor_url, json=failure_info)
 
 
 if __name__ == "__main__":

Copy link
Contributor

@cmtice cmtice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Suggested possibly adding a comment.

Created using spr 1.3.7
@boomanaiden154 boomanaiden154 merged commit 9161760 into main Oct 25, 2025
11 checks passed
@boomanaiden154 boomanaiden154 deleted the users/boomanaiden154/ci-make-premerge-advisor-upload-to-both-advisor-instances branch October 25, 2025 02:31
llvm-sync bot pushed a commit to arm/arm-toolchain that referenced this pull request Oct 25, 2025
So that we do not have to worry about synchronizing data between the two
clusters. This also enables this script to work on AArch64, although
we'll look at enabling that later.

Reviewers: cmtice

Reviewed By: cmtice

Pull Request: llvm/llvm-project#165058
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