Skip to content

Validate vLLM Router on GB200: DEP4, DEP8, and 1P/2D - #2549

Closed
cquil11 wants to merge 35 commits into
mainfrom
agent/vllm-router-gb200-smoke
Closed

cquil11 wants to merge 35 commits into
mainfrom
agent/vllm-router-gb200-smoke

Merge remote-tracking branch 'origin/main' into agent/vllm-router-gb2…

66ea40c
Select commit
Loading
Failed to load commit list.
Claude / Claude Code Review completed Aug 27, 2026 in 33m 13s

Code review found 2 new issues (4 earlier findings still open)

Found 7 candidates, confirmed 6: 2 posted in this review, 4 from earlier reviews still open. See review comments for details.

Details

Severity Count
🔴 Important 1
🟡 Nit 5
🟣 Pre-existing 0
Severity File:Line Issue
🔴 Important runners/launch_gb200-nv.sh:737 New vllm-router success gate exits before log bundling on gb200; cleanup_srt_job trap (lines 719-725, pre-existing) only
🟡 Nit perf-changelog.yaml:5751 New sweep config keys dsv4-fp4-b200-vllm-router-agentic-agg, dsv4-fp4-b200-vllm-router-agentic-disagg (configs/nvidia-ma