workflows: pin all actions to SHA#11908
Conversation
Signed-off-by: Patrick Stephens <pat@telemetryforge.io>
Signed-off-by: Patrick Stephens <pat@telemetryforge.io>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (9)
💤 Files with no reviewable changes (1)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (5)
📝 WalkthroughWalkthroughThis PR pins many GitHub Actions usages to explicit commit SHAs across build, test, release, lint, and cron workflows, adds a ChangesGitHub Actions Supply Chain Hardening
Estimated code review effort 🎯 4 (Complex) | ⏱️ ~45 minutes Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@lecaros related to #11846 (comment) |
Signed-off-by: Patrick Stephens <pat@telemetryforge.io>
Signed-off-by: Patrick Stephens <pat@telemetryforge.io>
Signed-off-by: Patrick Stephens <pat@telemetryforge.io>
Used
pinactto switch all actions to use pinned SHA values rather than release or, even worse, floatingmasterbranches.Replaced some unsupported actions with better ones that have SHA support:
Removed unused workflows to reduce maintenance and complexity burden:
The only outstanding one is the OSS fuzzer action:
google/oss-fuzz/infra/cifuzz/actions/run_fuzzers@master@DavidKorczynski can we switch it to using a specific SHA rather than relying on that floating
masterbranch? That way we can also enable the option to require pinned SHAs for this repo too.Enter
[N/A]in the box, if an item is not applicable to your change.Testing
Before we can approve your change; please submit the following in a comment:
If this is a change to packaging of containers or native binaries then please confirm it works for all targets.
ok-package-testlabel to test for all targets (requires maintainer to do).Documentation
Backporting
Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.
Summary by CodeRabbit