Skip to content

docs-build: switch to RTX Pro 6000, add changed-files filter - #2584

Open
jameslamb wants to merge 2 commits into
NVIDIA:mainfrom
jameslamb:docs-build-runners
Open

docs-build: switch to RTX Pro 6000, add changed-files filter#2584
jameslamb wants to merge 2 commits into
NVIDIA:mainfrom
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

Also noticed that there's a docs_build group in the changed-files configuration, but it isn't used to limit when docs builds runs. This adds that as well.

Notes for Reviewers

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

@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 WIP: docs-build: switch to RTX Pro 6000, add changed-files filter Sep 10, 2026
Comment thread .github/workflows/pr.yaml
pull-requests: read
secrets: inherit # zizmor: ignore[secrets-inherit]
uses: rapidsai/shared-workflows/.github/workflows/custom-job.yaml@release/26.10
if: fromJSON(needs.changed-files.outputs.changed_file_groups).build_docs

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.

This looked like an omission, given that there's configuration for it:

uses: rapidsai/shared-workflows/.github/workflows/changed-files.yaml@main
with:
files_yaml: |
build_docs:

This fixes that, so some types of PRs that don't affect the docs will not have a docs CI job.

Comment thread .github/workflows/pr.yaml
build_type: pull-request
matrix_name: docs-build
docs-build:
needs: [docs-build-matrix]

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.

I was surprised to not see any conda packages needed for the docs builds, but looks like that was an intentional change in #2067

@jameslamb jameslamb changed the title WIP: docs-build: switch to RTX Pro 6000, add changed-files filter docs-build: switch to RTX Pro 6000, add changed-files filter Sep 10, 2026
@jameslamb
jameslamb marked this pull request as ready for review September 10, 2026 02:53
@jameslamb
jameslamb requested a review from a team as a code owner September 10, 2026 02:53
@jameslamb
jameslamb requested a review from gforsyth September 10, 2026 02:53
@jameslamb
jameslamb requested review from a team as code owners September 10, 2026 13:32
@jameslamb
jameslamb changed the base branch from release/26.10 to main September 10, 2026 13:37
@jameslamb
jameslamb removed request for a team and KyleFromNVIDIA September 10, 2026 13:37
@jameslamb
jameslamb marked this pull request as draft September 10, 2026 13:41
@jameslamb
jameslamb marked this pull request as ready for review September 10, 2026 13:41
@jameslamb

Copy link
Copy Markdown
Member Author

/merge

@jameslamb

Copy link
Copy Markdown
Member Author

One C++ test failure:

[ RUN      ] AnnMGTest/AnnMGTestF_float.AnnMG/8
terminate called after throwing an instance of 'raft::cuda_error'
  what():  CUDA error encountered at: file=/tmp/conda-bld-output/bld/rattler-build_libcuvs-headers/host_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_/include/raft/util/cudart_utils.hpp line=139 function=void raft::copy(Type*, const Type*, size_t, rmm::_RMM_26_12::cuda_stream_view) [with Type = long int; size_t = long unsigned int]: call='cudaMemcpyAsync(dst, src, len * sizeof(Type), cudaMemcpyDefault, stream.get())', Reason=cudaErrorMisalignedAddress:misaligned address
Obtained 16 stack frames
#1 in /opt/conda/envs/test/bin/gtests/libcuvs/../../../lib/libcuvs.so(+0x6707f6) [0x7dc83781f7f6]
#2 in /opt/conda/envs/test/bin/gtests/libcuvs/../../../lib/libcuvs.so(+0x1c6d413) [0x7dc838e1c413]
#3 in /opt/conda/envs/test/bin/gtests/libcuvs/../../../lib/libcuvs.so(+0x1c6d69f) [0x7dc838e1c69f]
#4 in /opt/conda/envs/test/bin/gtests/libcuvs/../../../lib/libgomp.so.1: GOMP_parallel +0x46 [0x7dc82717ef36]
#5 in /opt/conda/envs/test/bin/gtests/libcuvs/../../../lib/libcuvs.so(+0x1c755d6) [0x7dc838e245d6]
#6 in ./../../..//bin/gtests/libcuvs/NEIGHBORS_MG_TEST(+0x70796) [0x5eed6ff16796]
#7 in ./../../..//bin/gtests/libcuvs/NEIGHBORS_MG_TEST(+0x9752c) [0x5eed6ff3d52c]
#8 in ./../../..//bin/gtests/libcuvs/NEIGHBORS_MG_TEST(+0x97801) [0x5eed6ff3d801]
#9 in ./../../..//bin/gtests/libcuvs/NEIGHBORS_MG_TEST(+0x97ba7) [0x5eed6ff3dba7]
#10 in ./../../..//bin/gtests/libcuvs/NEIGHBORS_MG_TEST(+0x98026) [0x5eed6ff3e026]
#11 in ./../../..//bin/gtests/libcuvs/NEIGHBORS_MG_TEST(+0x9eac3) [0x5eed6ff44ac3]
#12 in ./../../..//bin/gtests/libcuvs/NEIGHBORS_MG_TEST(+0x9efb1) [0x5eed6ff44fb1]
#13 in ./../../..//bin/gtests/libcuvs/NEIGHBORS_MG_TEST(+0x2eddf) [0x5eed6fed4ddf]
#14 in /usr/lib/x86_64-linux-gnu/libc.so.6(+0x2a601) [0x7dc826b5e601]
#15 in /usr/lib/x86_64-linux-gnu/libc.so.6: __libc_start_main +0x88 [0x7dc826b5e718]
#16 in ./../../..//bin/gtests/libcuvs/NEIGHBORS_MG_TEST(+0x2eece) [0x5eed6fed4ece]

CMake Error at run_gpu_test.cmake:26 (execute_process):
  execute_process failed command indexes:

    1: "Abnormal exit with child return code: Subprocess aborted"
....
The following tests FAILED:
	 32 - NEIGHBORS_MG_TEST (Failed)
/__w/cuvs/cuvs

(build link)

Re-trying.

cc @aamijar for your tracking

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