Skip to content

fix: nightly wheel naming for non-post versions#17538

Merged
Fridge003 merged 12 commits intomainfrom
nightly-wheel-naming
Jan 24, 2026
Merged

fix: nightly wheel naming for non-post versions#17538
Fridge003 merged 12 commits intomainfrom
nightly-wheel-naming

Conversation

@dougyster
Copy link
Copy Markdown
Collaborator

@dougyster dougyster commented Jan 22, 2026

Motivation

When building nightly wheels at an exact tag commit (Distance=0 from tag), setuptools-scm generates a version without a dev number (eg. 0.5.7 instead of 0.5.7.devN). This results in wheel filenames like sglang-0.5.7-py3-none-any.whl that are not unique, causing new builds to overwrite previous wheels with the same base version.

Modifications

Modified the nightly wheel build workflow to detect when building at an exact tag (distance=0) and force a dev0 version using the SETUPTOOLS_SCM_PRETEND_VERSION environment variable. This ensures every nightly wheel has a unique filename.

Accuracy Tests

https://github.com/sgl-project/whl/releases/tag/nightly-2026-01-24-ebafc5cf5

Benchmarking and Profiling

n/a

Checklist

Review Process

  1. Ping Merge Oncalls to start the PR flow. See the PR Merge Process.
  2. Get approvals from CODEOWNERS and other reviewers.
  3. Trigger CI tests with comments or contact authorized users to do so.
    • /tag-run-ci-label, /rerun-failed-ci, /tag-and-rerun-ci
  4. After green CI and required approvals, ask Merge Oncalls to merge.

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@github-actions github-actions Bot added the dependencies Pull requests that update a dependency file label Jan 22, 2026
Comment thread python/pyproject.toml Outdated
@Kangyan-Zhou
Copy link
Copy Markdown
Collaborator

https://github.com/sgl-project/whl/releases/tag/nightly-2026-01-22-232982a0d

Why do we have 5 files? Look at other nightly releases we only have 3 files

@dougyster
Copy link
Copy Markdown
Collaborator Author

https://github.com/sgl-project/whl/releases/tag/nightly-2026-01-22-232982a0d

Why do we have 5 files? Look at other nightly releases we only have 3 files

These are from previous workflow trials when I was attempting to fix this - I used the same commit to reproduce the bug so this commit's nightly wheel has more versions from my tests; once merged each nightly wheel will only have three files


# Build wheel - setuptools-scm automatically generates version from git tags
# Parse git describe output to detect exact tag builds (distance=0)
DESC=$(git describe --tags --long --match 'v*' 2>/dev/null || echo 'v0.0.0-0-g0000000')
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We need to keep the git describe command aligned:

https://github.com/sgl-project/sglang/blob/main/python/pyproject.toml#L167

The git describe command here will generate 0.5.6.post2 tag rather than 0.5.7

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thank you for the comment! Fixed~

@Fridge003 Fridge003 merged commit 58799d9 into main Jan 24, 2026
51 checks passed
@Fridge003 Fridge003 deleted the nightly-wheel-naming branch January 24, 2026 04:51
Johnsonms pushed a commit to Johnsonms/sglang that referenced this pull request Feb 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants