docs-build: switch to RTX Pro 6000 - #8555
Merged
Merged
Conversation
jameslamb
added
improvement
jameslamb
force-pushed
the
docs-builds
branch
from
September 10, 2026 01:21
dd2ff1a to
58fa13a
Compare
16 tasks
jameslamb
commented
Sep 10, 2026
| uses: rapidsai/shared-workflows/.github/workflows/custom-job.yaml@release/26.10 | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: ${{ fromJSON(needs.docs-build-matrix.outputs.matrix) }} |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
jameslamb
changed the title
jameslamb
commented
Sep 10, 2026
jameslamb
commented
Sep 10, 2026
This comment was marked as resolved.
This comment was marked as resolved.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
rapids-bot Bot
pushed a commit
to rapidsai/shared-workflows
that referenced
this pull request
Sep 10, 2026
…O 6000 (#627) 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: * rapidsai/cucim#1156 * rapidsai/cugraph-docs#229 * NVIDIA/cuml#8555 Authors: - James Lamb (https://github.com/jameslamb) Approvers: - Bradley Dice (https://github.com/bdice) URL: #627
This comment was marked as resolved.
This comment was marked as resolved.
jameslamb
marked this pull request as draft
September 10, 2026 13:38
jameslamb
changed the title
jameslamb
force-pushed
the
docs-builds
branch
from
September 10, 2026 14:38
27e07f6 to
603bf32
Compare
jameslamb
changed the title
jameslamb
marked this pull request as ready for review
September 10, 2026 15:08
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
bdice
approved these changes
Sep 10, 2026
Member
Author
|
/merge |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contributes to rapidsai/build-planning#333
Updates to hopefully reduce how often
docs-buildjobs get stuck waiting for CI runners.docs-buildmatrix to set job details