Skip to content

WIP: [NOT READY FOR REVIEW] enforce a floor on libnvjitlink, build wheels with CUDA 13.0.x, test wheels against mix of CTK versions#21671

Draft
jameslamb wants to merge 16 commits intorapidsai:mainfrom
jameslamb:test-older-ctk
Draft

WIP: [NOT READY FOR REVIEW] enforce a floor on libnvjitlink, build wheels with CUDA 13.0.x, test wheels against mix of CTK versions#21671
jameslamb wants to merge 16 commits intorapidsai:mainfrom
jameslamb:test-older-ctk

Conversation

@jameslamb
Copy link
Member

Description

Contributes to rapidsai/build-planning#257

  • builds CUDA 13 wheels with the 13.0 CTK
  • ensures wheels ship with a runtime dependency of nvidia-nvjitlink>={whatever-minor-version-they-were-built-against}

Contributes to rapidsai/build-planning#256

  • updates wheel tests to cover a range of CTK versions (we previously, accidentally, were only testing the latest 12.x and 13.x)

Other changes

  • ensures conda packages also take on floors of libnvjitlink>={whatever-version-they-were-built-against}

Notes for Reviewers

How I tested this

This uses wheels from similar PRs from RAPIDS dependencies, at build and test time:

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@jameslamb jameslamb added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Mar 5, 2026
@copy-pr-bot
Copy link

copy-pr-bot bot commented Mar 5, 2026

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
Copy link
Member Author

/ok to test

@jameslamb
Copy link
Member Author

/ok to test

@jameslamb
Copy link
Member Author

/ok to test

@jameslamb
Copy link
Member Author

jameslamb commented Mar 9, 2026

/merge

(edit) Ugh, meant to /ok to test but did this by memory. I've put a DO NOT MERGE label on this so we don't get an unexpected early merge

@github-actions github-actions bot added the pylibcudf Issues specific to the pylibcudf package label Mar 9, 2026
@jameslamb jameslamb added DO NOT MERGE Hold off on merging; see PR for details labels Mar 9, 2026
@jameslamb
Copy link
Member Author

/ok to test

RAPIDS_PY_WHEEL_NAME="${RAPIDS_PY_CUDA_SUFFIX}" RAPIDS_PY_WHEEL_PURE="1" rapids-get-pr-artifact --pkg_name distributed-ucxx ucxx 604 python wheel
)
LIBUCXX_WHEELHOUSE=$(
RAPIDS_PY_WHEEL_NAME="libucxx_${RAPIDS_PY_CUDA_SUFFIX}" rapids-get-pr-artifact ucxx 604 cpp wheel "${UCXX_COMMIT}"
Copy link
Member Author

@jameslamb jameslamb Mar 9, 2026

Choose a reason for hiding this comment

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

😬 I hit GitHub's rate limits

Downloading and decompressing ucxx_wheel_cpp_libucxx_cu12_x86_64 from Run ID into /tmp/tmp.FFhweajGIk

error fetching artifacts: HTTP 403: API rate limit exceeded for installation. If you reach out to GitHub Support for help, please include the request ID B4E0:1E3815:1337C1F:517B113:69AF40CF and timestamp 2026-03-09 21:51:11 UTC. For more on scraping GitHub and how it may affect your rights, please review our Terms of Service (https://docs.github.com/en/site-policy/github-terms/github-terms-of-service) (https://api.github.com/repositories/617172783/actions/artifacts?per_page=100&page=322)

(build link)

Look at the limits printed in that job... this repo was pushing it already:

    "core": {
      "limit": 15000,
      "used": 14483,
      "remaining": 517,
      "reset": 1773094676
    },

Seeing that ?page=322 in the message is concerning... nothing in our artifact-fetching should be paging over that many results!

I think the problem is that the run wasn't found. Notice "from Run ID into" (empty space!).

I'll put up a fix for that.

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixing that over here: rapidsai/gha-tools#249

@jameslamb
Copy link
Member Author

/ok to test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cudf.pandas Issues specific to cudf.pandas DO NOT MERGE Hold off on merging; see PR for details improvement Improvement / enhancement to an existing function non-breaking Non-breaking change pylibcudf Issues specific to the pylibcudf package Python Affects Python cuDF API.

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

1 participant