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.

Sign up for GitHub

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

Jump to bottom

Artifact llvm-fa57c7a6-almalinux-x64 contains install folder, which at least doubles its size. #5096

Closed
anmyachev opened this issue Nov 7, 2024 · 2 comments · Fixed by #5111
Closed

Artifact llvm-fa57c7a6-almalinux-x64 contains install folder, which at least doubles its size. #5096

anmyachev opened this issue Nov 7, 2024 · 2 comments · Fixed by #5111

Comments

@anmyachev
Copy link
Contributor

I noticed this because for the other commit: b74e588e - install folder is not there and the archive size is only about 1 gigabyte.

image
pbchekin added a commit to pbchekin/triton that referenced this issue Nov 11, 2024
@pbchekin
Fix llvm-build for almalinux
bd45c18 
Fixes triton-lang#5096
@pbchekin pbchekin mentioned this issue Nov 11, 2024
Merged
7 tasks
@pbchekin
Copy link
Contributor

Should be fixed in #5111, but the LLVM build needs to be re-triggered to upload a correct almalinux tarball.

@peterbell10
Copy link
Contributor

I've triggered a manual run, hopefully it doesn't mind overwriting existing artefacts

Luosuu pushed a commit to Luosuu/triton that referenced this issue Nov 13, 2024
@pbchekin @Luosuu
Fix llvm-build for almalinux (triton-lang#5111)
bdf1a19 
If a directory with LLVM exists in the runner (for example, a leftover
from a previous run for the same LLVM commit id), then the new LLVM
build will be copied to `install` subdirectory. To avoid this case we
should remove the existing LLVM installation directory.

Fixes triton-lang#5096.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix llvm-build for almalinux pbchekin/triton
3 participants
@pbchekin @peterbell10 @anmyachev