Skip to content

Fix zizmor ref-version-mismatch on the allowlist-check action pin - #73057

Merged
potiuk merged 1 commit into
v3-3-testfrom
fix-zizmor-allowlist-pin-v3-3-test
Sep 13, 2026
Merged

potiuk merged 1 commit into
v3-3-testfrom
fix-zizmor-allowlist-pin-v3-3-test

Conversation

@vatsrahul1001

Copy link
Copy Markdown
Contributor

After the ruff/mypy static-check restore (#73046), the 3.3.2 sync PR #72946 still had one static-check failure: zizmor's online ref-version-mismatch audit on .github/workflows/asf-allowlist-check.yml.

The apache/infrastructure-actions/allowlist-check action was pinned to a hash carrying a # main comment — a branch, not a version — which the audit flags. (Local/offline zizmor runs miss this; it only surfaces in CI's online run, which is why #73046 shipped with it still red.)

Fix: pin to the released allowlist-check/v1.0.0 tag, matching main (backport of the relevant line from #70623 "Pin apache/infrastructure-actions to its released tags").

Verified with the prek zizmor hook run online (GH_TOKEN=… prek run zizmor --all-files): Passed. This was the last static-check failure blocking a clean 3.3.2rc1 sync PR.

Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 4.8)

Generated-by: Claude Code (Opus 4.8) following the guidelines

The action was pinned to a hash carrying a '# main' comment, which zizmor's
online ref-version-mismatch audit flags (a branch is not a version). Pin to the
released allowlist-check/v1.0.0 tag, matching main (#70623). This was the last
static-check failure on the 3.3.2 sync PR after the ruff/mypy restore (#73046).
@vatsrahul1001 vatsrahul1001 added this to the Airflow 3.3.2 milestone Sep 13, 2026
@vatsrahul1001 vatsrahul1001 added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Sep 13, 2026
@vatsrahul1001 vatsrahul1001 changed the title Pin apache/infrastructure-actions allowlist-check to a released tag Fix zizmor ref-version-mismatch on the allowlist-check action pin Sep 13, 2026
@potiuk
potiuk merged commit 54b8fa4 into v3-3-test Sep 13, 2026
64 checks passed
@potiuk
potiuk deleted the fix-zizmor-allowlist-pin-v3-3-test branch September 13, 2026 03:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants