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

One-Off Backfill Tool for Managing GitHub Permissions as Code #4091

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Alaurant
Copy link

Description

This pull request introduces a one-off backfill tool that transfers GitHub permissions into a YAML file. This tool is designed to prepare for #3998, which focuses on managing GitHub teams' permissions as code. To ensure the data remains current, I will update this backfill tool just before both this PR and #3998 are ready to merge.

Discussion Points:

Auto-generation of YAML Files for Missing GitHub Teams:

During the backfill process, I have identified approximately 500 teams currently only on GitHub (You can view the detailed list here). Should I auto-generate YAML files for all these teams to avoid discrepancies and prevent overwriting GitHub team settings when #3998 goes live?

Auto-generation of YAML Files for Missing GitHub Special Teams:

Currently, our teams/ directory contains YAML files for only 5 out of about 50 special teams(A special team is a team in jenkinsci that has more than 1 repo permission) identified on GitHub. Should I create YAML files for the remaining teams to ensure we can manage other permissions in YAML format in the future?

Please share any thoughts or suggestions you have. I really appreciate your time and help.

@timja
Copy link
Member

timja commented Sep 14, 2024

Could you do this in stages instead of all 2000+ files at once?

i.e. could just do the special teams first? and a few plugins?

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