Skip to content

added the workflow and script to pin commit hash#6497

Closed
antedotee wants to merge 1 commit intopipe-cd:masterfrom
antedotee:add-commit-hash-workflow
Closed

added the workflow and script to pin commit hash#6497
antedotee wants to merge 1 commit intopipe-cd:masterfrom
antedotee:add-commit-hash-workflow

Conversation

@antedotee
Copy link
Copy Markdown
Contributor

What this PR does:
This PR introduces a workflow and a script which will ensure every tag is pinned to the commit hash. If anything uses tag and not a commit hash, the workflow will fail and will tell you to run hack/gha-reversemap.sh apply-reversemap locally. I have also added .gha-reversemap.yml which will act as a single source of truth for approved hashes.

Why we need it:
For security purposes, makes the supply chain auditable

Which issue(s) this PR fixes:

Fixes #6492

@antedotee antedotee requested a review from a team as a code owner February 2, 2026 05:54
Signed-off-by: antedotee <soniyadav2051982@gmail.com>
@antedotee antedotee force-pushed the add-commit-hash-workflow branch from 1fe0f43 to 575e52a Compare February 2, 2026 05:54
@antedotee
Copy link
Copy Markdown
Contributor Author

@khanhtc1202 Please take a look. Instead of raising PR solely for the workflow, the workflow was referencing the local script, so I added the local script here for end to end setup. I have run the script locally and it is working perfectly. If this gets merged, I think I can raise next PR for changing all the tags to commit hashes, and another PR for referencing the changes made in the documentation so that contributors can reference it to know what is going on.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 5, 2026

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the Stale label Mar 5, 2026
@Warashi
Copy link
Copy Markdown
Member

Warashi commented Mar 9, 2026

I think we should enforce pinning with the native GitHub feature, not implementing the custom solution.
https://docs.github.com/en/actions/reference/security/secure-use#using-third-party-actions

WDYT? @khanhtc1202

@github-actions github-actions Bot removed the Stale label Mar 10, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 9, 2026

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the Stale label Apr 9, 2026
@github-actions
Copy link
Copy Markdown
Contributor

This PR was closed because it has been stalled for 7 days with no activity. Feel free to reopen if still applicable.

@github-actions github-actions Bot closed this Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

There should be a commit hash check that ensures nothing uses tag and uses only commit hash

2 participants