Skip to content

Conversation

@beyhan
Copy link
Member

@beyhan beyhan commented Nov 10, 2025

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.

@beyhan beyhan requested review from a team, Gerg, cweibel, rkoster and stephanme and removed request for a team November 10, 2025 08:47
@beyhan beyhan added the toc label Nov 10, 2025
@beyhan beyhan force-pushed the add-cloudfoundry-tutorials branch 3 times, most recently from 0f7ccdf to 22c54e2 Compare November 10, 2025 08:53
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.
@beyhan beyhan force-pushed the add-cloudfoundry-tutorials branch from 22c54e2 to ee37a5d Compare November 10, 2025 09:20
@beyhan
Copy link
Member Author

beyhan commented Nov 10, 2025

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

Working Groups MUST only contain repos from one CFF Github Org. The org MUST be defined in the working group yaml.

Let's discuss our options during the TOC meeting tomorrow.

- name: Andreas Koppenhöfer
github: akop
- name: Jovan Kostovski
github: chombium
Copy link
Contributor

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.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done with: eed6d57

@anita-flegg anita-flegg self-requested a review November 10, 2025 16:53
Copy link
Member

@stephanme stephanme left a 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
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Inbox

Development

Successfully merging this pull request may close these issues.

4 participants