Skip to content

Conversation

@bdice
Copy link
Contributor

@bdice bdice commented Feb 6, 2025

This completes the migration to NVKS runners now that all libraries have been tested and rapidsai/shared-workflows#273 has been merged.

xref: https://github.com/rapidsai/build-infra/issues/184

@bdice bdice requested a review from a team as a code owner February 6, 2025 23:50
@bdice bdice requested a review from jameslamb February 6, 2025 23:50
@bdice bdice added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Feb 6, 2025
Copy link
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

A conda-cpp-build job failed here, with a network error.

./build.sh: line 19: hasArg: command not found
Building for the architecture of the GPU in the system...
[1/9] Creating directories for 'rapids-cmake-populate'
[1/9] Performing download step (download, verify and extract) for 'rapids-cmake-populate'
-- Downloading...
   dst='/opt/conda/conda-bld/work/examples/c/build/_deps/rapids-cmake-subbuild/rapids-cmake-populate-prefix/src/branch-25.04.zip'
   timeout='none'
   inactivity timeout='none'
-- Using src='https://github.com/rapidsai/rapids-cmake/archive/refs/heads/branch-25.04.zip'
CMake Error at rapids-cmake-subbuild/rapids-cmake-populate-prefix/src/rapids-cmake-populate-stamp/download-rapids-cmake-populate.cmake:163 (message):
  Each download failed!

    error: downloading 'https://github.com/rapidsai/rapids-cmake/archive/refs/heads/branch-25.04.zip' failed
          status_code: 22
          status_string: "HTTP response code said error"
          log:
          --- LOG BEGIN ---
          Host github.com:443 was resolved.

(build link)

I've restarted it. Unfortunately that job failing means it'll be a few hours until this is mergeable, because the C++ tests jobs here take 1-4 hours to run. (example recent build

@bdice
Copy link
Contributor Author

bdice commented Feb 7, 2025

/merge

@rapids-bot rapids-bot bot merged commit 8e6db92 into rapidsai:branch-25.04 Feb 7, 2025
69 checks passed
divyegala pushed a commit to divyegala/cuvs that referenced this pull request Feb 12, 2025
This completes the migration to NVKS runners now that all libraries have been tested and rapidsai/shared-workflows#273 has been merged.

xref: rapidsai/build-infra#184

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

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

URL: rapidsai#669
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