CI: bump CUDA image to 12.9.1#5107
Conversation
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@ustc.edu.cn>
There was a problem hiding this comment.
Pull request overview
This PR updates the CUDA container image version used in the CI workflow from 12.6.2 to 12.9.1, keeping the workflow current with newer CUDA releases.
- Updated NVIDIA CUDA Docker image from version 12.6.2 to 12.9.1
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
📝 WalkthroughWalkthroughUpdated the GitHub Actions CUDA test workflow: bumped the nvidia/cuda container image from 12.6.2-cudnn-devel-ubuntu22.04 to 12.9.1-cudnn-devel-ubuntu22.04 and added a step to install Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Possibly related PRs
Suggested labels
Suggested reviewers
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Repository UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (28)
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
.github/workflows/test_cuda.yml (1)
50-50: Update PaddlePaddle to cu129 wheels for CUDA 12.9.1.PaddlePaddle provides nightly wheels for CUDA 12.9 via the cu129 repository, while line 50 currently uses cu126 wheels (CUDA 12.6). The cu126 variant is compatible with LibCUDA 12.0 and above, but cu129 is recommended for CUDA 12.9. To ensure optimal compatibility with the CUDA 12.9.1 runtime, change the index URL from
cu126tocu129or confirm that cu126 wheels will not cause runtime issues in this environment.
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/workflows/test_cuda.yml(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (29)
- GitHub Check: Upload results
- GitHub Check: Test Python (6, 3.10)
- GitHub Check: Test Python (5, 3.10)
- GitHub Check: Test Python (4, 3.10)
- GitHub Check: Test Python (5, 3.13)
- GitHub Check: Test Python (6, 3.13)
- GitHub Check: Test Python (4, 3.13)
- GitHub Check: Test Python (1, 3.10)
- GitHub Check: Test Python (3, 3.13)
- GitHub Check: Test Python (3, 3.10)
- GitHub Check: Test Python (2, 3.13)
- GitHub Check: Test Python (2, 3.10)
- GitHub Check: Test Python (1, 3.13)
- GitHub Check: Build wheels for cp311-manylinux_x86_64
- GitHub Check: Build wheels for cp310-manylinux_aarch64
- GitHub Check: Build wheels for cp311-win_amd64
- GitHub Check: Build wheels for cp311-macosx_x86_64
- GitHub Check: Build wheels for cp311-macosx_arm64
- GitHub Check: Build C++ (cpu, cpu)
- GitHub Check: Build C++ (cuda120, cuda)
- GitHub Check: Build C++ (rocm, rocm)
- GitHub Check: Test C++ (false, false, false, true)
- GitHub Check: Build C++ (clang, clang)
- GitHub Check: Test C++ (false, true, true, false)
- GitHub Check: Test C++ (true, true, true, false)
- GitHub Check: Test C++ (true, false, false, true)
- GitHub Check: Build C library (2.18, libdeepmd_c.tar.gz)
- GitHub Check: Analyze (python)
- GitHub Check: Analyze (c-cpp)
🔇 Additional comments (1)
.github/workflows/test_cuda.yml (1)
22-22: Container image version update looks good.The CUDA image bump from 12.6.2 to 12.9.1 maintains the same base configuration (nvidia/cuda with cudnn on ubuntu22.04), and most downstream steps use generic
cu12tags that are compatible across CUDA 12.x versions.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## devel #5107 +/- ##
==========================================
+ Coverage 82.12% 82.14% +0.02%
==========================================
Files 709 709
Lines 72508 72458 -50
Branches 3616 3616
==========================================
- Hits 59545 59520 -25
+ Misses 11801 11776 -25
Partials 1162 1162 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@ustc.edu.cn>
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.