[CI] Add fields in CI matrix that determine devcontainer repo and runner labels - #11022
[CI] Add fields in CI matrix that determine devcontainer repo and runner labels#11022wmaxey wants to merge 2 commits into
Conversation
|
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 (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 10 remain after this review. 📝 WalkthroughSummary by CodeRabbit
WalkthroughGPU metadata now includes explicit GPU counts and configurable runner and image settings. Dispatch jobs emit operating system and GPU count fields. Linux and Windows workflows forward these values to job actions, which use them for GPU setup and passthrough. ChangesGPU runner configuration
Suggested reviewers: Merge Risk: ⚪ Minimal · up to This change adds configurable CI matrix inputs for devcontainer repositories and runner labels, with no actionable merge-blocking risk remaining beyond normal checks and review. Comment |
😬 CI Workflow Results🟥 Finished in 3h 19m: Pass: 99%/597 | Total: 20d 04h | Max: 3h 18m | Hits: 28%/2580244See results here. AI failure analysis1. CUDA Experimental launch config trait arguments rejected · 4 jobsExplanation: All four builds fail while compiling the same `launch_smoke.cu` assertions under both affected CUDA/GCC combinations and language standards. The logs do not include a deeper template diagnostic, but the failure is isolated to querying launch traits through `decltype` of the local CTAD-created `kernel` variable. Evidence: Copy this prompt into a coding agentJobs: 2. CUB ArgMinLastMax returns an earlier maximum index · 1 jobExplanation: The `abs_less_t` case for five million `short` inputs produces the expected maximum value but an earlier index instead of the expected last matching index. This points to incorrect last-maximum tie handling in the ArgMinLastMax reduction path rather than a test wrapper or CTest failure. Evidence: Copy this prompt into a coding agentJobs: 3. RAPIDS cugraph uses deleted RMM device_scalar rvalue constructor · 1 jobExplanation: Current cugraph passes temporary `size_t{0}` values to an RMM `device_scalar` overload that explicitly deletes rvalue initializers, causing the same compile error across many cugraph translation units. The other requested RAPIDS components built far enough for cugraph to be reported as the sole failing component. Evidence: Copy this prompt into a coding agentJobs: |
Description
closes
Allows switching which devcontainers and runner labels are used for testing. This gives us the ability to supply different inputs when generating test lists.
As a follow-up I'm going to change the windows devcontainer labels to follow the format of
cpp-{host}-{cuda}as this prevents us from further simplifying the devcontainer tag generation.rapidsai/devcontainers#758
Checklist