[ci][cpu] Update compiler to GCC-13 in jammy-aarch64#166849
[ci][cpu] Update compiler to GCC-13 in jammy-aarch64#166849fadara01 wants to merge 7 commits intogh/fadara01/7/basefrom
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/166849
Note: Links to docs will display an error until the docs builds have been completed. ❌ 2 New FailuresAs of commit cf99307 with merge base fbd70fb ( NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
robert-hardwick
left a comment
There was a problem hiding this comment.
Need to update GCC_VERSION variable
|
Actually, we also need to update both manylinux and jammy to GCC14 as per @malfet 's comment about manylinux standards on #166736:
Let's address the GCC version mismatch for AArch64 between jammy and manylinux first (and get jammy to build with GCC13), then we'll raise PRs to this stack updating both to GCC14, which I think is related to #149828 and #152426 |
|
Oh, OpenBLAS is failing to link with GCC-13 in jammy due to missing |
@fadara01 just install gcc13-gfortran or something (I bet there is a line somewhere in the scripts that does it already) |
Yup that's what I'm doing locally |
Alternative, you can just move CI to a more recent version of Ubuntu |
|
Current |
|
After rebasing, we now have a new failure from a GCC13 warning that's being treated as error: |
|
I think this is the corresponding gcc issue for the new warning we're seeing: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113239 let's silent it... |
|
@pytorchbot merge -f "AArch64 operator benchmarks are very flakey" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Merge failedReason: Command Details for Dev Infra teamRaised by workflow job |
|
Darn, after rebasing, we now fail with another GCC13 warning |
|
@pytorchbot merge --ignore-current "operator benchmarks are flakey" |
|
❌ 🤖 pytorchbot command failed: Try |
|
@pytorchbot merge --ignore-current |
Merge startedYour change will be merged while ignoring the following 2 checks: operator_benchmark / x86-opbenchmark-test / test (cpu_operator_benchmark_short, 1, 1, linux.12xlarge), operator_benchmark / aarch64-opbenchmark-test / test (cpu_operator_benchmark_short, 1, 1, linux.arm64.m8g.4xlarge) Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
This is needed because manylinux uses GCC-13 since #152825 As a result of the current compiler version mismatches, we've seen tests passing jammy-aarch64 pre-commit CI, but failing for wheels built in manylinux Related to: #166736 ghstack-source-id: a4d92fc Pull-Request: pytorch/pytorch#166849
Stack from ghstack (oldest at bottom):
This is needed because manylinux uses GCC-13 since #152825
As a result of the current compiler version mismatches, we've seen tests passing jammy-aarch64 pre-commit CI, but failing for wheels built in manylinux
Related to: #166736
cc @jgong5 @mingfeima @XiaobingSuper @sanchitintel @ashokei @jingxu10 @jerryzh168 @aditew01 @seemethere @malfet @pytorch/pytorch-dev-infra @snadampal @milpuz01 @nikhil-arm