Skip to content

Pin GitHub Actions to SHAs [SRE-1802]#353

Draft
tylergohl wants to merge 1 commit into
mainfrom
sre-1802-pin-actions-to-shas
Draft

Pin GitHub Actions to SHAs [SRE-1802]#353
tylergohl wants to merge 1 commit into
mainfrom
sre-1802-pin-actions-to-shas

Conversation

@tylergohl
Copy link
Copy Markdown

@tylergohl tylergohl commented May 21, 2026

Summary

Pin GitHub Actions references in this repo to immutable commit SHAs (with the original tag/branch preserved in a trailing comment). Part of SRE-1802 — org-wide supply-chain hardening.

Scope

This PR pins:

  • Internal ctrliq/* reusable workflow and action references
  • External actions not handled by clo-ciq's parallel Node 20 effort

No actions in this repo were claimed by clo-ciq's effort — this PR pins everything.

Files touched

  • .github/workflows/devel_images.yml

Test plan

  • Workflow runs succeed on this branch (or on next trigger after merge)
  • Once this and any companion clo-ciq PR merge, this repo has zero unpinned external/internal refs and sha_pinning_required can be enabled on it

Copilot AI review requested due to automatic review settings May 21, 2026 01:32
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 supply chain by pinning previously tag-referenced third-party actions to immutable commit SHAs in the development image build workflow, aligning with the org-wide action pinning effort (SRE-1802).

Changes:

  • Pinned Kaven-Universe/github-action-current-date-time from @v1 to a commit SHA (retaining the version as an inline comment).
  • Pinned benjlevesque/short-sha from @v3.0 to a commit SHA (retaining the version as an inline comment).

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

Comment on lines +58 to +60
uses: Kaven-Universe/github-action-current-date-time@7c1d8465d1bfd7d35466d2ee82a8e460780561ae # v1

- uses: benjlevesque/short-sha@v3.0
- uses: benjlevesque/short-sha@599815c8ee942a9616c92bcfb4f947a3b670ab0b # v3.0
@cigamit cigamit added SECURITY A security related issue like a CVE specifically build labels May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build SECURITY A security related issue like a CVE specifically

Development

Successfully merging this pull request may close these issues.

3 participants