-
Notifications
You must be signed in to change notification settings - Fork 228
Add CF Tutorials to CF org automation #1364
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
base: main
Are you sure you want to change the base?
Conversation
0f7ccdf to
22c54e2
Compare
This prs is adding the `cloudfoundry-tutorials` GitHub org to the CF org automation. Additionally, the pr proposes people who should interest to contribute to the tutorials as reviewers.
22c54e2 to
ee37a5d
Compare
|
We explicitly state in https://github.com/cloudfoundry/community/blob/main/toc/rfc/rfc-0036-multiple-github-orgs.md#working-groups-are-per-cff-github-org
Let's discuss our options during the TOC meeting tomorrow. |
| - name: Andreas Koppenhöfer | ||
| github: akop | ||
| - name: Jovan Kostovski | ||
| github: chombium |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@beyhan , please add me so that:
1 - I can fix typos, etc.
2 - I can keep the info organization/architecture up-to-date
3 - I know when things are being changed so that I know when to publish.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done with: eed6d57
stephanme
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The org automation won't work because Doc WG now contains repos from different github orgs. This breaks the team generation as described in [rfc-0014-github-teams-and-access.md|https://github.com/cloudfoundry/community/blob/main/toc/rfc/rfc-0014-github-teams-and-access.md]. Teams are local to a github org and it is not possible to grant repo access using teams of other orgs.
I should implement a test to let the validation fail for such a setup (currently, the resulting peribolos file is broken). IIRC, it was not possible so far to validate the repo org because of some exceptions here and there.
We could try to weaken this rule and e.g. allow different github orgs per WG area (but all repos within an area must be in the same org). Requires some changes to the team rules, e.g. it is not possible anymore to have a wg-[WORKING-GROUP-NAME] team with all reviewers/approvers but we can have such teams only per github org (i.e. there are 2 wg-docs teams: on in cloudfoundry and one in cloudfoundry-tutorial.
I had to work around a similar problem for wg-leads. Here I require that WG leads for WGs in other github orgs onboard additionally in the cloudfoundry org so that I can add them to the wg-leads team. This should work out because WG lead anyway have to maintain the WG charter in the community project.
| - name: Arsalan Khan | ||
| github: asalan316 | ||
| repositories: | ||
| - cloudfoundry-tutorials/cf4devs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently you can't have repos from different managed github orgs in one WG. The github org is defined on WG level (default: cloudfoundry). Validation seems to have room for improvement.
This prs is adding the
cloudfoundry-tutorialsGitHub org to the CF org automation. Additionally, the pr proposes people who should interest to contribute to the tutorials as reviewers.