Skip to content

[CI] Keep custom sgl-kernel wheel in CUDA CI#24291

Merged
Kangyan-Zhou merged 1 commit into
sgl-project:mainfrom
mmangkad:fix-ci-custom-sgl-kernel-wheel
May 3, 2026
Merged

[CI] Keep custom sgl-kernel wheel in CUDA CI#24291
Kangyan-Zhou merged 1 commit into
sgl-project:mainfrom
mmangkad:fix-ci-custom-sgl-kernel-wheel

Conversation

@mmangkad
Copy link
Copy Markdown
Contributor

@mmangkad mmangkad commented May 3, 2026

Summary

Fixes a regression from #21247 where CUDA CI always force-reinstalled sglang-kernel from the public wheel index, which could overwrite a freshly built PR sgl-kernel wheel when CUSTOM_BUILD_SGL_KERNEL=true.

Keep the custom-built wheel in that path so CI actually tests kernel changes from the PR.

See https://github.com/sgl-project/sglang/actions/runs/25264102281/job/74077130931?pr=24281#step:6:380.

cc @Kangyan-Zhou

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the ci_install_dependency.sh script to conditionally install the sglang-kernel. It introduces the CUSTOM_BUILD_SGL_KERNEL environment variable, which, when set to true, prevents the script from overwriting a freshly built kernel wheel with one from the remote index. I have no feedback to provide.

@Kangyan-Zhou
Copy link
Copy Markdown
Collaborator

/tag-and-rerun-ci

@github-actions github-actions Bot added the run-ci label May 3, 2026
@Kangyan-Zhou Kangyan-Zhou merged commit b7d8ceb into sgl-project:main May 3, 2026
117 of 136 checks passed
@Kangyan-Zhou
Copy link
Copy Markdown
Collaborator

@mmangkad thanks for the fix!

@mmangkad mmangkad deleted the fix-ci-custom-sgl-kernel-wheel branch May 3, 2026 05:00
LucQueen pushed a commit to LucQueen/sglang that referenced this pull request May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants