[CI] release-whl-kernel: clean root-owned build artifacts before checkout#23747
Merged
Kangyan-Zhou merged 1 commit intomainfrom Apr 26, 2026
Merged
[CI] release-whl-kernel: clean root-owned build artifacts before checkout#23747Kangyan-Zhou merged 1 commit intomainfrom
Kangyan-Zhou merged 1 commit intomainfrom
Conversation
…kout Self-hosted kernel-build nodes retain the workspace across jobs. Prior builds leave root-owned files under sgl-kernel/build/ (the in-container compile runs as root and writes to the bind-mounted workspace), which makes the next actions/checkout@v4 fail with EACCES on rmdir sgl-kernel/build/.cmake. Add a pre-checkout step to all four kernel-build jobs (build-cu129-matrix, build-cu130-matrix, build-rocm-matrix, build-musa43) that wipes the workspace via a throwaway alpine root container — same pattern already in nightly-72-gpu-gb200.yml. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Kangyan-Zhou
requested review from
Fridge003,
bingxche,
ispobock and
merrymercy
as code owners
Contributor
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
vguduruTT
pushed a commit
to vguduruTT/sglang
that referenced
this pull request
May 2, 2026
…kout (sgl-project#23747) Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
release-whl-kernel.yml(build-cu129-matrix,build-cu130-matrix,build-rocm-matrix,build-musa43) run on self-hosted nodes that retain the workspace across jobs. Their in-container compile runs as root and leaves root-owned files undersgl-kernel/build/, which causes the next run'sactions/checkout@v4to fail withEACCES: permission denied, rmdir 'sgl-kernel/build/.cmake'.alpine:3root container — same pattern already in use innightly-72-gpu-gb200.yml:132-140for the same reason.pr-test.yml's kernel-build jobs already have a workingsudo rm -rf $GITHUB_WORKSPACE/*cleanup, so they are intentionally left unchanged.release-whl-kernel.ymlwas the only kernel-building workflow with no cleanup at all.Test plan
release-whl-kernel.yml(workflow_dispatch) on a PR build and confirm the new "Clean workspace" step runs and the subsequentactions/checkout@v4succeeds with noEACCESerrors.wheel-python3.10-cuda12.9*,wheel-python3.10-cuda13.0*,wheel-python3.10-rocm{700,720},wheel-python3.10-musa43) as before.🤖 Generated with Claude Code