Skip to content

Fix release run lookup for annotated tags#2147

Open
fallintoplace wants to merge 1 commit into
NVIDIA:mainfrom
fallintoplace:fix-release-tag-run-lookup
Open

Fix release run lookup for annotated tags#2147
fallintoplace wants to merge 1 commit into
NVIDIA:mainfrom
fallintoplace:fix-release-tag-run-lookup

Conversation

@fallintoplace
Copy link
Copy Markdown

Fixes #2146.

lookup-run-id was resolving --tag inputs with git rev-parse <tag>, which returns the tag object SHA for annotated tags. That works for lightweight tags, but it breaks the documented release flow because cuda.core release tags are created with git tag -s, so the workflow can end up querying GitHub Actions with the wrong SHA.

This change peels tags to their underlying commit before calling gh run list, and adds a focused regression test that uses an annotated tag plus a fake gh shim to prove we pass the commit SHA through.

Checks:

  • bash -n ci/tools/lookup-run-id
  • uv run --with ruff ruff check ci/tools/tests/test_lookup_run_id.py
  • uv run --with pytest pytest --noconftest ci/tools/tests/test_lookup_run_id.py

@copy-pr-bot
Copy link
Copy Markdown
Contributor

copy-pr-bot Bot commented May 27, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions github-actions Bot added the CI/CD CI/CD infrastructure label May 27, 2026
@fallintoplace fallintoplace marked this pull request as ready for review May 27, 2026 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD CI/CD infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: release run lookup fails for signed/annotated tags

1 participant