Repository navigation
Conversation
The old psap-b200-mlperf cluster no longer exists. All B200 runs (accelerator=B200 in CSV) should route to the b200-pirate-vllm-dcgm dashboard. Also removes dead B200_PIRATE entry since no CSV row ever has accelerator=B200_PIRATE. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe B200 Grafana dashboard mapping now points to the pirate dashboard. The separate B200_PIRATE mapping is removed. ChangesB200 Grafana Dashboard
Estimated code review effort: 1 (Trivial) | ~5 minutes Suggested reviewers: Merge Risk: ⚪ Minimal · up to B200 links now target the pirate Grafana dashboard. No broken-link or metrics regression is established; staging checks remain outstanding but do not indicate a merge-blocking risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #102 +/- ##
======================================
Coverage ? 3.48%
======================================
Files ? 8
Lines ? 8329
Branches ? 0
======================================
Hits ? 290
Misses ? 8039
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
we will still need the old mlperf grafana dashboard, since 3.4-gA runs were done on that cluster. |
Summary
GRAFANA_DASHBOARDS["B200"]to point tob200-pirate-vllm-dcgminstead of the oldpsap-b200-mlperfdashboard (cluster no longer exists)B200_PIRATEentry — no CSV row ever hasaccelerator=B200_PIRATEsince Forge producesaccelerator=B200viagpu_type.upper()Context
All 4042 B200 runs in the CSV have
accelerator=B200. The routing was pointing to the oldpsap-b200-mlperfGrafana dashboard which has no metrics. The correct dashboard isb200-pirate-vllm-dcgm.Test plan
b200-pirate-vllm-dcgm🤖 Generated with Claude Code
Summary by CodeRabbit