feat(ci): Pin action versions to commit SHAs#5876
Conversation
Signed-off-by: Ayush More <ayushmore42595@gmail.com>
|
@benjdlambert plz review |
awanlin
left a comment
There was a problem hiding this comment.
Thanks for these changes @Ayushmore1214, spotted some typos and a pair of missed items.
I think we also should make changes to this file so that we automatically stay up to date with changes:
community-plugins/.github/renovate.json
Lines 1 to 10 in e6bd3fc
We should be able to just add "helpers:pinGitHubActionDigests" to the extends array to enable that.
Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com>
Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com>
Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com>
Signed-off-by: Ayush More <ayushmore42595@gmail.com>
|
@awanlin thank you for your review , I have made the changes respectively |
Thank you for this , just wondering do you think this should also be added to the Renovate config in the main backstage/backstage repository for same security benefitts? |
awanlin
left a comment
There was a problem hiding this comment.
Thanks for the follow up @Ayushmore1214, left one new comment to address.
Signed-off-by: Ayush More <ayushmore42595@gmail.com>
|
Done |
Signed-off-by: Ayush More <ayushmore42595@gmail.com>
awanlin
left a comment
There was a problem hiding this comment.
Awesome, let's get this merged @Ayushmore1214 🚀
This reverts commit 0ec0afa. Signed-off-by: Andre Wanlin <awanlin@spotify.com>
|
We ended up having to revert this change as the SHA's were not correct: #6117. Given this I'd prefer to let Renovate eventually generate the PR for these changes. |
* feat(ci): Pin action versions to commit SHAs Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Update .github/workflows/auto-version-bump-scheduler.yml Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Update .github/workflows/auto-version-bump-scheduler.yml Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Update .github/workflows/deprecate-archived-plugins.yml Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com> * fixed the typos Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Remove helper for pinning GitHub Action digests Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Fix formatting in renovate.json Signed-off-by: Ayush More <ayushmore42595@gmail.com> --------- Signed-off-by: Ayush More <ayushmore42595@gmail.com> Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Max Ramer <maxjramer@gmail.com>
backstage#6117) This reverts commit 0ec0afa. Signed-off-by: Andre Wanlin <awanlin@spotify.com> Signed-off-by: Max Ramer <maxjramer@gmail.com>
* feat(ci): Pin action versions to commit SHAs Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Update .github/workflows/auto-version-bump-scheduler.yml Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Update .github/workflows/auto-version-bump-scheduler.yml Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Update .github/workflows/deprecate-archived-plugins.yml Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com> * fixed the typos Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Remove helper for pinning GitHub Action digests Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Fix formatting in renovate.json Signed-off-by: Ayush More <ayushmore42595@gmail.com> --------- Signed-off-by: Ayush More <ayushmore42595@gmail.com> Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Max Ramer <maxjramer@gmail.com>
backstage#6117) This reverts commit 0ec0afa. Signed-off-by: Andre Wanlin <awanlin@spotify.com> Signed-off-by: Max Ramer <maxjramer@gmail.com>
* feat(ci): Pin action versions to commit SHAs Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Update .github/workflows/auto-version-bump-scheduler.yml Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Update .github/workflows/auto-version-bump-scheduler.yml Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Update .github/workflows/deprecate-archived-plugins.yml Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com> * fixed the typos Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Remove helper for pinning GitHub Action digests Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Fix formatting in renovate.json Signed-off-by: Ayush More <ayushmore42595@gmail.com> --------- Signed-off-by: Ayush More <ayushmore42595@gmail.com> Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Max Ramer <maxjramer@gmail.com>
backstage#6117) This reverts commit 0ec0afa. Signed-off-by: Andre Wanlin <awanlin@spotify.com> Signed-off-by: Max Ramer <maxjramer@gmail.com>
|
Ohhh , Thanks for getting it merged ! |
* feat(ci): Pin action versions to commit SHAs Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Update .github/workflows/auto-version-bump-scheduler.yml Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Update .github/workflows/auto-version-bump-scheduler.yml Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Update .github/workflows/deprecate-archived-plugins.yml Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com> Signed-off-by: Ayush More <ayushmore42595@gmail.com> * fixed the typos Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Remove helper for pinning GitHub Action digests Signed-off-by: Ayush More <ayushmore42595@gmail.com> * Fix formatting in renovate.json Signed-off-by: Ayush More <ayushmore42595@gmail.com> --------- Signed-off-by: Ayush More <ayushmore42595@gmail.com> Co-authored-by: Andre Wanlin <67169551+awanlin@users.noreply.github.com>
backstage#6117) This reverts commit 0ec0afa. Signed-off-by: Andre Wanlin <awanlin@spotify.com>
Hey, I just made a Pull Request!
Making our CI/CD a little more secure this pins all actions to specific commit sha, preventing unexpected breaks or supply-chain risk
✔️ Checklist
Signed-off-byline in the message. (more info)