Skip to content

HDDS-16816. Pin first-party actions to a major version tag - #108

Merged
adoroszlai merged 1 commit into
apache:latestfrom
adoroszlai:HDDS-16816
Oct 17, 2026
Merged

adoroszlai merged 1 commit into
apache:latestfrom
adoroszlai:HDDS-16816

Conversation

@adoroszlai

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

The GitHub Actions Policy requires a specific git hash only for external actions; actions in the apache/*, github/* and actions/* namespaces may be used without that restriction. Pinning those to a major version tag rather than to a hash lets a patch or minor release arrive without a pull request, which removes a large share of the Dependabot traffic a project sees. External actions still MUST be pinned to a specific git hash. (source)

  • This PR updates workflows to pin actions/* and apache/* by major version instead of SHA, as recommended by Apache Infra above. github/* actions are not used here.
  • Configure zizmor to allow such pinning.
  • Make sure zizmor workflow is triggered by changes in zizmor config.

https://issues.apache.org/jira/browse/HDDS-16816

How was this patch tested?

CI: https://github.com/adoroszlai/ozone-docker/actions/runs/38909025981
zizmor: https://github.com/adoroszlai/ozone-docker/actions/runs/38909026262
asf-allowlist-check: https://github.com/adoroszlai/ozone-docker/actions/runs/38909026075

@adoroszlai adoroszlai self-assigned this Oct 16, 2026

@smengcl smengcl left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks @adoroszlai for working on this.

@adoroszlai
adoroszlai merged commit 1a86f02 into apache:latest Oct 17, 2026
3 checks passed
@adoroszlai
adoroszlai deleted the HDDS-16816 branch October 17, 2026 06:04
@adoroszlai

Copy link
Copy Markdown
Contributor Author

Thanks @smengcl for the review.

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.

2 participants