Skip to content

GitHub: restrict runnable actions#2646

Draft
AP-Hunt wants to merge 5 commits intomainfrom
github_restrict_runnable_actions
Draft

GitHub: restrict runnable actions#2646
AP-Hunt wants to merge 5 commits intomainfrom
github_restrict_runnable_actions

Conversation

@AP-Hunt
Copy link
Member

@AP-Hunt AP-Hunt commented Aug 22, 2025

What

We'd like to restrict the set of actions that can be run by GitHub Actions to a number of trusted namespaces, and a smaller number of trusted specific actions pinned at specific known-good hashes.

We can't do that all at once without breaking things.

This PR introduces some new configuration options in repos.yml which allow us to opt a repository in to restricted actions, and define the trusted actions.

How to review

Review each commit in order. They tell a story and build up the picture. To review the changes repos.yml, compare the changes with the list of repositories in #1937

…ation

A subsequent commit will use this configuration option to set restrictions on
what GitHub Actions can be used in each repository
"github_actions_sets" will, in a subsequent commit, act like the
"access_levels" property. It will contain some reusbale, shared lists of GitHub
Actions to restrict repositories to using, so that we're not repeating the list
all the time.
…itories

A subsequent commit will configure each of Platform Engineering's repositories
to use this list
A subsequent commit will make use of these config values in the Terraform code.
For now, we're only setting it up for Platform Engineering repositories.
@AP-Hunt AP-Hunt requested a review from a team as a code owner August 22, 2025 12:28
@AP-Hunt AP-Hunt force-pushed the github_restrict_runnable_actions branch 3 times, most recently from daa51b8 to 22cd3d1 Compare August 22, 2025 13:59
@AP-Hunt AP-Hunt force-pushed the github_restrict_runnable_actions branch from 22cd3d1 to f8a066a Compare August 22, 2025 14:06
@kentsanggds
Copy link
Contributor

Parking this PR pending Andys return and further discussion on it

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