CI: Migrate custom jobs to RTX PRO 6000 - #23878
Conversation
|
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review. 📝 WalkthroughSummary by CodeRabbit
WalkthroughChangesGPU-based jobs in the build, sanitizer, Pandas, pull request, and test workflows now select RTX PRO 6000 runners. Packaged Java tests select RTX PRO 6000 for GPU runner migration
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This PR updates CI jobs to use RTX PRO 6000 runners while retaining L4 for ARM64; no actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.) ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
CI runtime comparison: L4 vs. RTX PRO 6000Updated baseline: the earlier version of this report used PR #23869, which ran before dependency caching was enabled. This comparison instead uses the cached L4 run from PR #23770 (run 33193794748). It started only 35 minutes before this PR's RTX PRO 6000 run and overlapped it in wall-clock time. Both runs include the I compared that L4 run with the completed RTX PRO 6000 jobs from this PR's previous commit,
Summary
Same-run matrix comparisonAs a tighter cross-check, run 33196496195 contains five shared-workflow matrix jobs with both AMD64 L4 and RTX PRO 6000 entries. This holds the commit, cache configuration, and run timing constant. All entries use the latest driver; CUDA, Python, OS, and dependency variants differ as shown.
RTX PRO 6000 was faster in all five matrix groups. Giving each matrix equal weight, mean job time was 34m 30s on L4 versus 10m 11s on RTX PRO 6000, a 70.5% reduction (3.39× speedup). Per-matrix reductions ranged from 22.3% to 85.9%. This same-run result avoids the cache and temporal confound in the original cross-PR comparison, but it still does not isolate GPU hardware: the matrix intentionally varies CUDA, Python, OS, and, for the wheel jobs, dependency age. In particular, the L4 wheel entries use oldest dependencies while the RTX PRO 6000 entries use latest dependencies. This is still a single observational before/after sample, not an isolated hardware benchmark. The aggregate is dominated by unusually long L4 docs, notebook, and Java jobs, so the 42.4% aggregate reduction should not be interpreted as a general hardware speedup. Although the runs overlapped and used the same cache setup, PR source differences, cache-hit state, network behavior, and runner contention remain uncontrolled. Multiple paired runs would be needed to estimate stable hardware-specific speedups. |
|
/merge |
Description
Migrates custom CI jobs from L4 to RTX PRO 6000 runners.
The packaged-Java matrix retains L4 for ARM64 because RTX PRO 6000 runners are only available for AMD64.
Checklist