Skip to content

fix(ci): pin pypa/gh-action-pypi-publish to SHA for Apache allowlist#518

Merged
tqchen merged 2 commits intoapache:mainfrom
junrushao:junrushao/2026-03-30/fix-publish-wheel-action-pin
Mar 31, 2026
Merged

fix(ci): pin pypa/gh-action-pypi-publish to SHA for Apache allowlist#518
tqchen merged 2 commits intoapache:mainfrom
junrushao:junrushao/2026-03-30/fix-publish-wheel-action-pin

Conversation

@junrushao
Copy link
Copy Markdown
Member

Summary

Files changed

  • .github/workflows/publish_wheel.yml
  • .github/workflows/publish_orcjit_wheel.yml
  • .github/workflows/torch_c_dlpack.yml

Test plan

  • Verify the "Publish wheel" workflow no longer fails with startup_failure
  • Confirm all three publish workflows pass CI

Pin `pypa/gh-action-pypi-publish` from `@release/v1` (branch ref) to
`@ed0c53931b1dc9bd32cbe73a98c7f6766f8a527e` (v1.13.0 SHA) in all three
publish workflows: `publish_wheel.yml`, `publish_orcjit_wheel.yml`, and
`torch_c_dlpack.yml`.

This fixes the `startup_failure` in the "Publish wheel" GitHub Actions
workflow (https://github.com/apache/tvm-ffi/actions/runs/23773754182)
caused by Apache's org-level action allowlist policy requiring
SHA-pinned third-party actions. The same class of fix was previously
applied in d8bd189 (apache#507) for `astral-sh/setup-uv` and
`pypa/cibuildwheel`.
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

…ake 4.3.1

- Pin pypa/gh-action-pypi-publish from @release/v1 (branch ref) to
  @ed0c53931b1dc9bd32cbe73a98c7f6766f8a527e (v1.13.0) in all three
  publish workflows, fixing startup_failure caused by Apache's
  org-level action allowlist.
- Exclude cmake==4.3.1 in dev dependency group: that release has
  incomplete wheels on PyPI (missing manylinux x86_64), breaking
  `uv sync --group dev` in CI lint jobs.

Refs: https://github.com/apache/tvm-ffi/actions/runs/23773754182
"ipython",
"cython>=3.0",
"cmake",
"cmake!=4.3.1",
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@tqchen tqchen merged commit 61f90eb into apache:main Mar 31, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants