Skip to content

[backport] Remove rapidsai from channel resolution to avoid rapids-xgboost (#221) - #225

Merged
chyunsu3 merged 1 commit into
rapidsai:release/26.10from
chyunsu3:xgboost_fix_2610
Sep 14, 2026
Merged

chyunsu3 merged 1 commit into
rapidsai:release/26.10from
chyunsu3:xgboost_fix_2610

Conversation

@chyunsu3

Copy link
Copy Markdown
Contributor

Backport #221 to the 26.10 branch.

cc @bdice

@chyunsu3
chyunsu3 requested a review from a team as a code owner September 12, 2026 21:51
@chyunsu3
chyunsu3 requested a review from bdice September 12, 2026 21:51
@github-actions github-actions Bot added the conda Relates to conda packaging label Sep 12, 2026
@chyunsu3 chyunsu3 changed the title Remove rapidsai from channel resolution to avoid rapids-xgboost (… [backport] Remove rapidsai from channel resolution to avoid rapids-xgboost (#221) Sep 12, 2026
@chyunsu3 chyunsu3 added bug Something isn't working non-breaking Introduces a non-breaking change ci labels Sep 12, 2026
@coderabbitai

coderabbitai Bot commented Sep 12, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: c547ecdb-9db8-4189-9840-43dec182d1da

📥 Commits

Reviewing files that changed from the base of the PR and between a885a9d and ee52c12.

📒 Files selected for processing (9)
  • conda/environments/all_cuda-129_arch-aarch64.yaml
  • conda/environments/all_cuda-129_arch-x86_64.yaml
  • conda/environments/all_cuda-133_arch-aarch64.yaml
  • conda/environments/all_cuda-133_arch-x86_64.yaml
  • conda/environments/clang_tidy_cuda-129_arch-x86_64.yaml
  • conda/environments/clang_tidy_cuda-133_arch-x86_64.yaml
  • conda/environments/cpp_all_cuda-129_arch-x86_64.yaml
  • conda/environments/cpp_all_cuda-133_arch-x86_64.yaml
  • dependencies.yaml
💤 Files with no reviewable changes (9)
  • conda/environments/clang_tidy_cuda-133_arch-x86_64.yaml
  • conda/environments/clang_tidy_cuda-129_arch-x86_64.yaml
  • conda/environments/cpp_all_cuda-129_arch-x86_64.yaml
  • conda/environments/all_cuda-129_arch-aarch64.yaml
  • conda/environments/all_cuda-133_arch-x86_64.yaml
  • dependencies.yaml
  • conda/environments/cpp_all_cuda-133_arch-x86_64.yaml
  • conda/environments/all_cuda-129_arch-x86_64.yaml
  • conda/environments/all_cuda-133_arch-aarch64.yaml

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.


📝 Summary

Summary by CodeRabbit

  • Chores
    • Updated Conda environment configurations to use the rapidsai-nightly channel instead of rapidsai.
    • Retained conda-forge where previously configured.
    • Applied the channel updates across supported CUDA versions, architectures, and development environments.

Walkthrough

The pull request updates Conda channel configuration in eight environment files and dependencies.yaml. Each change replaces or removes rapidsai while retaining rapidsai-nightly and any existing conda-forge entry.

Changes

Conda channel update

Layer / File(s) Summary
Dependency channel list
dependencies.yaml
The dependency channel list removes rapidsai and retains rapidsai-nightly.
Environment channel lists
conda/environments/*
CUDA and clang-tidy environment files replace or remove rapidsai while retaining other configured channels.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to ee52c

The dependency source and generated environments now use aligned channel lists, with no actionable merge risk identified.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed 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…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The description states that this pull request backports PR #221 to the 26.10 branch. This matches the channel-resolution changes in the pull request.
Title check ✅ Passed The title clearly identifies the removal of rapidsai from channel resolution and states the reason. It matches the main change in the pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@chyunsu3

This comment was marked as outdated.

@chyunsu3

Copy link
Copy Markdown
Contributor Author

I'm going to admin-merge this, to unblock the CI. (Right now, the CI jobs for the 26.10 branch are not running past the Conda build stage.) I will be closely watching the CI to address any issues that arise.

@chyunsu3
chyunsu3 merged commit 4654629 into rapidsai:release/26.10 Sep 14, 2026
70 checks passed
@chyunsu3
chyunsu3 deleted the xgboost_fix_2610 branch September 14, 2026 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci conda Relates to conda packaging non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant