Skip to content

compute-matrix: add 'docs-build' matrix, switch docs builds to RTX PRO 6000 - #627

Merged
rapids-bot[bot] merged 4 commits into
mainfrom
ci/docs-build-matrix
Sep 10, 2026
Merged

compute-matrix: add 'docs-build' matrix, switch docs builds to RTX PRO 6000#627
rapids-bot[bot] merged 4 commits into
mainfrom
ci/docs-build-matrix

Conversation

@jameslamb

@jameslamb jameslamb commented Sep 3, 2026

Copy link
Copy Markdown
Member

Contributes to rapidsai/build-planning#333

Introduces a docs-build matrix to the compute-matrix job, so we can centrally control which runner types are used for docs builds.

Notes for Reviewers

Why RTX Pro 6000?

There's less competition for those runners than for L4s, see rapidsai/build-planning#333 for details.

Why centralize here?

Centralizing that choice of runner type would mean that changing it in the future wouldn't require PRs like NVIDIA/cudf#23878 in every repo. I think that's justified here... it's really unlikely that docs builds would be sensitive to the particular details of a runner, as long as it's a GPU RAPIDS supports.

Notes for reviewers

How I tested this

Saw passing builds and the expected behavior on all of these:

@jameslamb jameslamb added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Sep 3, 2026
Comment thread .github/workflows/compute-matrix.yaml Outdated
Comment thread .github/workflows/compute-matrix.yaml Outdated
Comment thread .github/workflows/compute-matrix.yaml Outdated
# - some GPU with low queue
# - latest driver
#
- { ARCH: 'amd64', PY_VER: '3.11', CUDA_VER: '12.9.2', LINUX_VER: 'rockylinux8', GPU: 'rtxpro6000', DRIVER: 'latest'}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like to use some version of CUDA 13 for our docs builds. Python version is less important. We have better coverage of the environments for CUDA 13 and fewer packaging oddities as CUDA 12 gets harder to support.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense!

Most of the docs jobs are already using rapidsai/ci-conda:26.10-latest (which is CUDA 13) so I'll just match that.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did that in 74c50ad

@bdice bdice changed the title WIP: compute-matrix: add 'docs-build' matrix, switch docs builds to RTX Pro 6000 WIP: compute-matrix: add 'docs-build' matrix, switch docs builds to RTX PRO 6000 Sep 4, 2026
@jameslamb
jameslamb changed the base branch from main to release/26.10 September 10, 2026 01:20
@jameslamb jameslamb changed the title WIP: compute-matrix: add 'docs-build' matrix, switch docs builds to RTX PRO 6000 compute-matrix: add 'docs-build' matrix, switch docs builds to RTX PRO 6000 Sep 10, 2026
@jameslamb
jameslamb marked this pull request as ready for review September 10, 2026 03:32
@jameslamb
jameslamb requested a review from a team as a code owner September 10, 2026 03:32
@jameslamb
jameslamb requested review from bdice and msarahan and removed request for a team and msarahan September 10, 2026 03:32
@jameslamb
jameslamb changed the base branch from release/26.10 to main September 10, 2026 12:38
@jameslamb

Copy link
Copy Markdown
Member Author

I've re-targeted this to main, based on rapidsai/rmm#2542 (comment)

@jameslamb

Copy link
Copy Markdown
Member Author

/merge

@rapids-bot
rapids-bot Bot merged commit 85122ca into main Sep 10, 2026
2 checks passed
@jameslamb
jameslamb deleted the ci/docs-build-matrix branch September 10, 2026 13:24
rapids-bot Bot pushed a commit to rapidsai/rmm that referenced this pull request Sep 10, 2026
Contributes to rapidsai/build-planning#333

Updates to hopefully reduce how often `docs-build` jobs get stuck waiting for CI runners.

* switches those jobs to RTX Pro 6000 runners
* uses shared `docs-build` matrix to set job details

## Notes for Reviewers

CI will pass here once rapidsai/shared-workflows#627 is merged.

Authors:
  - James Lamb (https://github.com/jameslamb)

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

URL: #2542
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.

2 participants