Skip to content

[Klaud Cold] dsv4-fp4-b200-vllm: nightly-700d39b image, prefill-schedule-interval 16 at CONC≥256 / 更新 B200 DSv4 镜像并在高并发下启用 prefill 调度间隔 - #2534

Open
xinli-sw wants to merge 16 commits into
mainfrom
b200_agg_dsv4
Open

[Klaud Cold] dsv4-fp4-b200-vllm: nightly-700d39b image, prefill-schedule-interval 16 at CONC≥256 / 更新 B200 DSv4 镜像并在高并发下启用 prefill 调度间隔#2534
xinli-sw wants to merge 16 commits into
mainfrom
b200_agg_dsv4

Conversation

@xinli-sw

@xinli-sw xinli-sw commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Bump dsv4-fp4-b200-vllm image to vllm/vllm-openai:nightly-700d39b55813fce6fe5339b334beb55d2a39f598
  • Add --prefill-schedule-interval 16 for CONC >= 256 in dsv4_fp4_b200_vllm.sh

中文说明

  • dsv4-fp4-b200-vllm 镜像升级至 nightly-700d39b55813fce6fe5339b334beb55d2a39f598
  • dsv4_fp4_b200_vllm.sh 中,当 CONC >= 256 时启用 --prefill-schedule-interval 16

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Thanks for the contribution! Please reach out to respective companies' CODEOWNER to fill in the latest PR_REVIEW_CHECKLIST.md before pinging core maintainer on Slack for review. In order for the signoff PR check bot to trigger, you must follow the PR_REVIEW_CHECKLIST.md template correctly, including the phrase As a PR reviewer and CODEOWNER, I have reviewed this and have.

For PR verification, add the full-sweep-fail-fast label (strongly recommended) to this PR — the benchmark sweep only runs on labeled PRs. Use full-sweep-enabled only if you need matrix jobs to keep running past a failure.

PR authors are responsible for ensuring that after merging, all GitHub Action jobs fully pass. A lot of the time, failures are just flakes and simply re-running the failed jobs will fix it. See GitHub's docs on re-running failed jobs


感谢你的贡献!请联系相应公司的 CODEOWNER 填写最新的 PR_REVIEW_CHECKLIST.md,然后再在 Slack 上联系核心维护者进行审阅。为了触发 signoff PR 检查机器人,你必须正确遵循 PR_REVIEW_CHECKLIST.md 模板,包括保留英文语句 As a PR reviewer and CODEOWNER, I have reviewed this and have

如需进行 PR 验证,请为此 PR 添加 full-sweep-fail-fast 标签(强烈推荐)— 基准测试 sweep 仅在带有标签的 PR 上运行。仅当需要矩阵任务在失败后继续运行时才使用 full-sweep-enabled

PR 作者有责任确保合并后所有 GitHub Action 任务完全通过。 很多时候失败只是偶发抖动(flake),重新运行失败的任务即可解决。参见 GitHub 关于重新运行失败任务的文档

@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.

LGTM — straightforward image bump and a scoped CLI flag addition, consistent with prior recipe-tuning PRs in this repo.

Extended reasoning...

Overview

This PR touches a single benchmark recipe (dsv4_fp4_b200_vllm.sh), its master-config image tag (configs/nvidia-master.yaml), and the changelog (perf-changelog.yaml). It bumps the vLLM image to a nightly build and adds an unconditional --prefill-schedule-interval 16 override for CONC >= 256, which takes precedence over the existing DP_ATTENTION-only interval of 4. This mirrors the standard 'nuke'-style image bump + perf-tuning pattern used throughout this repo's recipe files.

Security risks

None. This is a benchmark shell script and YAML config for internal perf sweeps — no user input, auth, or credential handling is touched. The image tag change points to an official vllm/vllm-openai nightly build referenced by immutable digest-like tag.

Level of scrutiny

Low. This is a narrow, mechanical change confined to a single model/hardware/engine recipe combination, following the same shape as many prior merged PRs (image bump + a single new conditional CLI flag). The bug-hunting system found no issues, and the one candidate concern raised (the CONC>=256 branch silently overriding the DP_ATTENTION interval-4 setting) was examined and ruled out — it reads as an intentional override for high-concurrency prefill scheduling, not an unintended clobber, and only affects this one recipe's own script.

Other factors

No outstanding reviewer comments to address. Change is fully self-contained (script + config + changelog kept in sync), and verification is enforced via the full-sweep-fail-fast label already applied, which will run the actual benchmark sweep against this recipe before merge.

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

@xinli-sw xinli-sw added WIP and removed WIP labels Aug 9, 2026
@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown
Contributor

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

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant