Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix LLVM Project Tests Workflow on Linux (#122221) #275

Merged
merged 4 commits into from
Jan 17, 2025

Conversation

philippjh
Copy link
Member

@philippjh philippjh commented Jan 16, 2025

This patch fixes the LLVM project tests workflow on Linux. Two changes were needed. Firstly, some commands need to be performed with sudo now that the container executes as a non-root user. Second, we needed to change from ubuntu-latest to ubuntu-22.04 as ubuntu-latest not defaults to ubuntu-24.04 which causes setup-python to install a python executable linked against a newer version of glibc that is not found on ubuntu 22.04, which causes failures when CMake cannot execute the python interpreter that it finds.

Cherry-picked from upstream commit: a75917679549109fcbf92cb63ef61638517713d6

boomanaiden154 and others added 4 commits January 16, 2025 15:08
This patch fixes the LLVM project tests workflow on Linux. Two changes
were needed. Firstly, some commands need to be performed with sudo now
that the container executes as a non-root user. Second, we needed to
change from `ubuntu-latest` to `ubuntu-22.04` as `ubuntu-latest` not
defaults to `ubuntu-24.04` which causes `setup-python` to install a
python executable linked against a newer version of glibc that is not
found on ubuntu 22.04, which causes failures when CMake cannot execute
the python interpreter that it finds.
@philippjh philippjh marked this pull request as ready for review January 17, 2025 09:47
@philippjh philippjh closed this Jan 17, 2025
@philippjh philippjh reopened this Jan 17, 2025
@stephenneuendorffer stephenneuendorffer merged commit 614da1d into aie-public Jan 17, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants