Skip to content

chore: bump srt-slurm to v2.49.0 - #3882

Merged
cquil11 merged 1 commit into
mainfrom
chore/bump-srt-slurm-v2.49.0
Oct 9, 2026
Merged

cquil11 merged 1 commit into
mainfrom
chore/bump-srt-slurm-v2.49.0

Conversation

@cquil11

@cquil11 cquil11 commented Oct 9, 2026

Copy link
Copy Markdown
Collaborator

Update inferencex-e2e/utils/srt-slurm from v2.47.2 (9d65c0ae) to v2.49.0 (7d983c79). The range adds the llm-d router frontend (NVIDIA/srt-slurm#588), a detailed_trace_annotation profiling knob (#589), and scrapes SGLang attention-DP followers for custom AIPerf metrics (#554).

Signed-off-by: Cam Quilici <cjquilici@gmail.com>
@cquil11
cquil11 requested a review from a team October 9, 2026 20:10
@cquil11
cquil11 merged commit 5895b96 into main Oct 9, 2026
5 of 6 checks passed
@cquil11
cquil11 deleted the chore/bump-srt-slurm-v2.49.0 branch October 9, 2026 20:15

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good — straightforward submodule version bump.

What was reviewed: confirmed the diff is exactly a one-line gitlink update for inferencex-e2e/utils/srt-slurm (9d65c0a → 7d983c7), with no other files touched. No perf-changelog.yaml entry accompanies this bump, but checked recent entries and found no established pattern of adding one for pure submodule pointer bumps (as opposed to in-repo config/recipe changes), so this isn't flagged as a blocking gap.

Extended reasoning...

The only change is a Git submodule pointer bump (srt-slurm 9d65c0a → 7d983c7), with no other files in the InferenceX repo modified — no source, config, test, or docs content changed. No security-sensitive surface (auth, crypto, permissions) is touched. The submodule's own internal diff can't be inspected from this shallow, unchecked-out checkout, but mechanical version bumps of this kind are low-risk and match the PR description. No outstanding review comments or CHANGES_REQUESTED exist in the timeline, and the change is small and self-contained enough that a human need not block on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant