Skip to content

docs-build: make use of 'cpu8' runners explicit - #540

Merged
rapids-bot[bot] merged 5 commits into
rapidsai:release/26.10from
jameslamb:docs-build-runners
Sep 10, 2026
Merged

docs-build: make use of 'cpu8' runners explicit#540
rapids-bot[bot] merged 5 commits into
rapidsai:release/26.10from
jameslamb:docs-build-runners

Conversation

@jameslamb

@jameslamb jameslamb commented Sep 10, 2026

Copy link
Copy Markdown
Member

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

After initially opening this to switch the type of GPU runner used (as I'm doing across the rest of RAPIDS), I realize that this repo's actually using CPU-only runners for its docs builds.

That wasn't obvious because it's implicitly relying on the default value here: https://github.com/rapidsai/shared-workflows/blob/e9a2657b6a56aae9f8fe2ef4bc8e613fd79438d4/.github/workflows/custom-job.yaml#L41

This PR makes that explicit.

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

copy-pr-bot Bot commented Sep 10, 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: docs-build: switch to RTX Pro 6000 docs-build: switch to RTX Pro 6000 Sep 10, 2026
@jameslamb
jameslamb marked this pull request as ready for review September 10, 2026 02:59
@jameslamb
jameslamb requested a review from a team as a code owner September 10, 2026 02:59
@jameslamb
jameslamb requested a review from gforsyth September 10, 2026 02:59
@greptile-apps

greptile-apps Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Greptile code reviews should be reviewed for accuracy and completeness. Please remember to give Greptile feedback (👍/👎) and consider leaving comments explaining why Greptile's feedback was helpful or not. This will help improve future code reviews.

RetriggerConfidence Score: 5/5

The PR appears safe to merge with no substantive issues identified.

Summary

  • Keeps the existing AMD64 architecture and CI container.
  • Makes CPU runner selection explicit for the shared custom-job workflow.

Reviews (2) · Last reviewed commit: "make use of CPU runners explicit"

@jameslamb jameslamb changed the title docs-build: switch to RTX Pro 6000 docs-build: make use of 'cpu8' runners explicit Sep 10, 2026
@jameslamb

Copy link
Copy Markdown
Member Author

/merge

@rapids-bot
rapids-bot Bot merged commit e6496a0 into rapidsai:release/26.10 Sep 10, 2026
77 checks passed
@jameslamb
jameslamb deleted the docs-build-runners branch September 10, 2026 13:40
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