Skip to content

Pin github actions to specific SHA-s - #19

Merged
haahaakon merged 9 commits into
mainfrom
fix/pin-gh-actions-to-sha
Jun 19, 2026
Merged

Pin github actions to specific SHA-s#19
haahaakon merged 9 commits into
mainfrom
fix/pin-gh-actions-to-sha

Conversation

@haahaakon

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR hardens the repository’s GitHub Actions workflows by pinning third-party actions to immutable commit SHAs (reducing supply-chain risk from tag drift), and improves tooling installation reproducibility in CI.

Changes:

  • Pinned actions/checkout, actions/setup-go, actions/cache, docker/* and github/codeql-action/* workflow dependencies to specific commit SHAs.
  • Replaced Helm installation via curl | bash with a pinned azure/setup-helm action and an explicit Helm version.
  • Added SHA-256 verification for the downloaded yq tarball in the release workflow.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/vulnerability-scan.yml Pins checkout + Go setup actions to SHAs for the scheduled vulnerability scan job.
.github/workflows/test-chart.yml Pins checkout + Go setup actions and switches Helm install to pinned azure/setup-helm with a fixed Helm version.
.github/workflows/release.yaml Pins build/release-related actions (checkout, Go, Docker actions, Helm) and adds yq checksum verification.
.github/workflows/codeql.yml Pins CodeQL workflow actions (checkout, init, analyze) to SHAs.
.github/workflows/build-and-test.yaml Pins checkout, Go setup, and cache actions to SHAs for CI build/test and security scan jobs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/test-chart.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Comment thread .github/workflows/vulnerability-scan.yml
Comment thread .github/workflows/test-chart.yml
Comment thread .github/workflows/release.yaml Outdated
Comment thread .github/workflows/build-and-test.yaml
haahaakon and others added 2 commits June 19, 2026 13:01
tar xz < file can be non-portable / fail depending on the tar implementation because stdin isn’t guaranteed to be used as the archive without -f. Use tar -xzf <file> (or tar -xzf - < file) to reliably extract the downloaded archive.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@haahaakon
haahaakon marked this pull request as ready for review June 19, 2026 11:29
@haahaakon
haahaakon merged commit b0f3886 into main Jun 19, 2026
7 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.

2 participants