Skip to content

ci: add selective sdist build plumbing - #2465

Merged
kkraus14 merged 1 commit into
NVIDIA:mainfrom
kkraus14:agent/selective-ci-sdists
Aug 13, 2026
Merged

ci: add selective sdist build plumbing#2465
kkraus14 merged 1 commit into
NVIDIA:mainfrom
kkraus14:agent/selective-ci-sdists

Conversation

@kkraus14

@kkraus14 kkraus14 commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Important

Stack dependency: PR 2 of 4. Builds on #2464, which has merged. #2466 and #2467 depend on this PR.

Merge order: #2464 (merged) -> #2465 (this PR) -> #2466 -> #2467. All branches live in the kkraus14 fork and target upstream main.

What

  • Add default-on package build inputs to both sdist reusable workflows.
  • Skip package-specific setup and sdist builds when that package is not selected.
  • Hydrate unchanged cuda-pathfinder and cuda-bindings wheels from the current CI run when a selected downstream sdist needs them.
  • Integrate selective downloads with ci: constrain internal builds to exact local wheels #2510's exact local-wheel constraints, without PIP_PRE or PIP_FIND_LINKS.
  • Preserve the current Linux and Windows compiler, cache, CTK, timeout, and path behavior.

Why

Selective wheel builds are not enough on their own: the sdist smoke jobs also rebuild every package today. This plumbing lets the final orchestration PR apply the same dependency closure to sdist coverage.

Behavior before activation

Every new input defaults to true, and this PR does not change any caller. Sdist coverage remains unchanged until the orchestration PR supplies selective values.

Validation

  • Full pre-commit hooks for both workflows
  • YAML and GitHub Actions lint
  • Review of all build-flag combinations and Linux/Windows artifact dependencies
  • Verified the final diff contains only the two sdist workflows on top of merged ci: add selective wheel build plumbing #2464/current main

Related to #299.

@kkraus14 kkraus14 added this to the cuda.bindings next milestone Jul 31, 2026
@kkraus14 kkraus14 added enhancement Any code-related improvements CI/CD CI/CD infrastructure labels Jul 31, 2026
@copy-pr-bot

copy-pr-bot Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

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.

@kkraus14
kkraus14 force-pushed the agent/selective-ci-sdists branch 2 times, most recently from 04c3eed to 72b6f6d Compare July 31, 2026 16:22
@github-actions github-actions Bot added the cuda.core Everything related to the cuda.core module label Jul 31, 2026
@kkraus14
kkraus14 force-pushed the agent/selective-ci-sdists branch 4 times, most recently from 1b9e3c3 to 3140b8f Compare August 3, 2026 16:23
@kkraus14
kkraus14 force-pushed the agent/selective-ci-sdists branch from 3140b8f to ba96295 Compare August 12, 2026 04:15
@kkraus14 kkraus14 self-assigned this Aug 12, 2026
@kkraus14
kkraus14 marked this pull request as ready for review August 12, 2026 04:16
@github-actions

This comment has been minimized.

@mdboom mdboom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@kkraus14
kkraus14 merged commit c78c596 into NVIDIA:main Aug 13, 2026
288 of 295 checks passed
@github-actions

Copy link
Copy Markdown
Doc Preview CI
Preview removed because the pull request was closed or merged.

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

Labels

CI/CD CI/CD infrastructure cuda.core Everything related to the cuda.core module enhancement Any code-related improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants