Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support a diverse set of owners while limiting blast radius #126

Open
BigLep opened this issue Jun 6, 2024 · 1 comment
Open

Support a diverse set of owners while limiting blast radius #126

BigLep opened this issue Jun 6, 2024 · 1 comment

Comments

@BigLep
Copy link
Contributor

BigLep commented Jun 6, 2024

Problem

Github orgs with a diverse set of projects and stakeholders (e.g., filecoin-project) have inherent risks by using github-mgmt/github-as-code. They ideally want a diverse-enough set of representatives with push access to filecoin-project/github-mgmt, but anyone who has push access can effectively make large permissions changes on projects that are unrelated to their own. It would be ideal for example if lotus maintainers could approve permissions changes to their repos or teams but not to other groups' repos and teams.

Ideas

  1. If instead of having one large .yaml file there were multiple YAML files (one per repo or team), codeowners could be used and branch protection to require PR approval from a codeowner.
  2. If gihtub-mgmt CI checks were intelligent to identify which resource or repo was modified, they could wait to pass until someone with write access to that repo or team approved the PR.
@BigLep
Copy link
Contributor Author

BigLep commented Jul 20, 2024

This issue was referenced in filecoin-project/community#710

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

No branches or pull requests

1 participant