Skip to content

Conversation

@cpcloud
Copy link
Contributor

@cpcloud cpcloud commented Jan 15, 2026

Putting this up for review in case there is some case I am missing, but I do not see anything that indicates we need to care about the version, even in artifact download. AFAIS there is only a single wheel produce per cuda version for a given run-id.

@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Jan 15, 2026

Auto-sync is disabled for ready for review pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@cpcloud cpcloud requested a review from leofang January 15, 2026 19:08
@cpcloud cpcloud force-pushed the remove-build-ctk-version-from-doc-builds branch from e37bf8e to e9ce825 Compare January 15, 2026 19:18
@cpcloud
Copy link
Contributor Author

cpcloud commented Jan 15, 2026

/ok to test

echo "CUDA_CORE_ARTIFACT_NAME=${CUDA_CORE_ARTIFACT_BASENAME}-${FILE_HASH}" >> $GITHUB_ENV
echo "CUDA_CORE_ARTIFACTS_DIR=$(realpath "$REPO_DIR/cuda_core/dist")" >> $GITHUB_ENV
CUDA_BINDINGS_ARTIFACT_BASENAME="cuda-bindings-python${PYTHON_VERSION_FORMATTED}-cuda${{ inputs.build-ctk-ver }}-linux-64"
CUDA_BINDINGS_ARTIFACT_BASENAME="cuda-bindings-python${PYTHON_VERSION_FORMATTED}-cuda*-linux-64"
Copy link
Member

Choose a reason for hiding this comment

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

Ah ha! So we needed the CTK ver also to infer the artifact name... hmmm... I will need to sleep over this.

@leofang leofang added enhancement Any code-related improvements P0 High priority - Must do! CI/CD CI/CD infrastructure labels Jan 16, 2026
@leofang leofang added this to the cuda.core beta 12 milestone Jan 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD CI/CD infrastructure enhancement Any code-related improvements P0 High priority - Must do!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants