Skip to content

ci: use proxy cache for pypi.anaconda.org, remove pip indices from dependencies.yaml - #244

Merged
rapids-bot[bot] merged 1 commit into
rapidsai:mainfrom
jameslamb:ci/pip-index
Oct 6, 2026
Merged

rapids-bot[bot] merged 1 commit into
rapidsai:mainfrom
jameslamb:ci/pip-index

Conversation

@jameslamb

Copy link
Copy Markdown
Member

Contributes to rapidsai/build-planning#338

Enables NVIDIA's self-hosted pull-through proxy cache for pypi.anaconda.org. See the linked issue for benefits of doing that.

  • removes --extra-index-url pointing at nightly index from dependencies.yaml (already set globally in CI images)
  • sets PIP_EXTRA_INDEX_URL explicitly in pip devcontainers (so it doesn't need to rely on values from dependencies.yaml)

@jameslamb jameslamb added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Oct 6, 2026
@copy-pr-bot

copy-pr-bot Bot commented Oct 6, 2026

Copy link
Copy Markdown

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.

@jameslamb jameslamb changed the title WIP: ci: use proxy cache for pypi.anaconda.org, remove pip indices from dependencies.yaml ci: use proxy cache for pypi.anaconda.org, remove pip indices from dependencies.yaml Oct 6, 2026
@jameslamb
jameslamb marked this pull request as ready for review October 6, 2026 18:47
@jameslamb
jameslamb requested review from a team as code owners October 6, 2026 18:47
@jameslamb
jameslamb requested a review from bdice October 6, 2026 18:47
@coderabbitai

coderabbitai Bot commented Oct 6, 2026

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: rapidsai/nvforest/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Enterprise
  • Run ID: 2be63520-f504-4622-9a8a-a61087f31f5e
📥 Commits

Reviewing files that changed from the base of the PR and between 949bf82 and eda863d.

📒 Files selected for processing (3)
  • .devcontainer/cuda12.9-pip/devcontainer.json
  • .devcontainer/cuda13.3-pip/devcontainer.json
  • dependencies.yaml
💤 Files with no reviewable changes (1)
  • dependencies.yaml

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


📝 Summary

Summary by CodeRabbit

  • Development Environment
    • CUDA 12.9 and CUDA 13.3 pip development containers now use the RAPIDS nightly wheels index for package installation.
  • Dependency Configuration
    • Generated requirements for selected RAPIDS packages no longer include the nightly wheels index as an extra package source.

Walkthrough

The CUDA 12.9 and 13.3 pip devcontainers now set PIP_EXTRA_INDEX_URL to the RAPIDS nightly wheels index. Five dependency groups no longer add that index to requirements output.

Changes

Pip index configuration

Layer / File(s) Summary
Configure nightly wheel index
.devcontainer/cuda*-pip/devcontainer.json, dependencies.yaml
The CUDA 12.9 and 13.3 pip devcontainers set PIP_EXTRA_INDEX_URL. Five dependency groups remove the requirements-output extra index URL.

Priority: ⬇️ Low

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

Change: Other

Merge Risk: ⚪ Minimal · up to eda86

The nightly index remains available in the supported devcontainer and CI install paths, so this change presents no actionable merge-blocking risk.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the proxy-cache change and removal of pip indices from dependencies.yaml.
Description check ✅ Passed The description explains the proxy-cache objective and the related changes to dependencies.yaml and pip devcontainers.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@jameslamb

Copy link
Copy Markdown
Member Author

/merge

@rapids-bot
rapids-bot Bot merged commit 7d7c1a7 into rapidsai:main Oct 6, 2026
60 checks passed
@jameslamb
jameslamb deleted the ci/pip-index branch October 6, 2026 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants