Skip to content

feat(conda): build noarch python packages separately - #20613

Merged
rapids-bot[bot] merged 6 commits into
NVIDIA:mainfrom
gforsyth:conda_noarch
Nov 18, 2025
Merged

feat(conda): build noarch python packages separately#20613
rapids-bot[bot] merged 6 commits into
NVIDIA:mainfrom
gforsyth:conda_noarch

Conversation

@gforsyth

Copy link
Copy Markdown
Contributor

Part of rapidsai/build-planning#43

Builds all of the noarch: python compatible conda packages as noarch packages and without the build fanout.

@gforsyth
gforsyth requested review from a team as code owners November 13, 2025 21:07
@gforsyth
gforsyth requested a review from AyodeAwe November 13, 2025 21:07
@gforsyth gforsyth added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Nov 13, 2025
Comment thread .github/workflows/pr.yaml Outdated
Comment thread ci/test_notebooks.sh Outdated
Comment thread ci/test_python_common.sh Outdated
@gforsyth

Copy link
Copy Markdown
Contributor Author

Hey @bdice -- are these known CI failures? I don't think I've touched wheel builds at all

@bdice

bdice commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

@gforsyth I think these failures have been resolved upstream, I merged in main.

@vyasr

vyasr commented Nov 18, 2025

Copy link
Copy Markdown
Contributor

@gforsyth I don't think there's any urgency here, so we can just go ahead and merge into main rather than moving to 25.12, right? Assuming so, CI is passing here so feel free to merge away in the morning.

@gforsyth

Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot
rapids-bot Bot merged commit 8fb2fa0 into NVIDIA:main Nov 18, 2025
138 checks passed
@gforsyth
gforsyth deleted the conda_noarch branch November 18, 2025 14:03
gforsyth added a commit that referenced this pull request Nov 20, 2025
xref rapidsai/shared-workflows#459
xref #20654
xref #20613

Currently these noarch builds have a dependency on the major cuda
version, so we need to fan out the builds by cuda major. Potential
followup here to make the builds properly pure (removing dependency on
`cuda-version`)
rapids-bot Bot pushed a commit that referenced this pull request Dec 5, 2025
Initially in #20613 I wasn't fanning out the conda noarch builds by `cuda_major` and so ran into artifact name collisions. I fixed that in #20678 but the remnant of the collision avoidance remains in the artifact names.

This non-standard name (compared to the other noarch artifacts) can break various helper tools for downloading artifacts from PRs into CI in other RAPIDS projects.

Removing the extra `noarch` in the artifact name here should sort that out.

xref rapidsai/build-planning#43

Authors:
  - Gil Forsyth (https://github.com/gforsyth)

Approvers:
  - Bradley Dice (https://github.com/bdice)

URL: #20794
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants