Skip to content

Commit 5054900

Browse files
[CI] Skip Running Prmeerge Advisor on AArch64
They were still running because the conditional was not correct. This patch fixes that so they do not interefere with the results of the job.
1 parent 6f5c8fe commit 5054900

File tree

3 files changed

+5
-2
lines changed

3 files changed

+5
-2
lines changed

.ci/premerge_advisor_explain.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ def main(
147147

148148
# Skip looking for results on AArch64 for now because the premerge advisor
149149
# service is not available on AWS currently.
150-
if platform.machine() == "arm64":
150+
if platform.machine() == "arm64" or platform.machine == "aarch64":
151151
sys.exit(0)
152152

153153
main(

.ci/premerge_advisor_upload.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ def main(commit_sha, workflow_run_number, build_log_files):
5959

6060
# Skip uploading results on AArch64 for now because the premerge advisor
6161
# service is not available on AWS currently.
62-
if platform.machine() == "arm64":
62+
if platform.machine() == "arm64" or platform.machine() == "aarch64":
6363
sys.exit(0)
6464

6565
main(args.commit_sha, args.workflow_run_number, args.build_log_files)

.github/workflows/premerge.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,9 @@ jobs:
5757
# container and non-container jobs.
5858
shell: bash
5959
steps:
60+
- name: Print platform
61+
run: |
62+
python3 -c "import platform; print(platform.machine())"
6063
- name: Checkout LLVM
6164
uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
6265
with:

0 commit comments

Comments
 (0)