Community: Decentralize membership lists

Created on 23 Oct 2019  路  14Comments  路  Source: open-telemetry/community

At the moment we list approvers/maintainers of all SIGs in the central file community-members.md in this repo. That means adding/editing those lists requires the attention of the admins/TC, while the decisions to promote member are actually made locally in the SIGs.

I suggest replacing the explicit lists with links to the respective repo and maintaining the lists there, to better scale up the process.

Similarly, the SIG information (schedule of meetings, etc.) can be also moved from the README in this repo to the READMEs in the respective repos. This repo will only have the links to information.

To avoid creating a bunch of refactoring work right now (unless someone wants to take it on), I suggest we do it incrementally as new requests (like #230) to change membership arrive.

enhancement

All 14 comments

We actually discussed it while back and @bogdandrutu @sarahnovotny and @bhs we onboard. It is a good idea to make it in stages.

Removing the "proposal" from title.

I like the idea, but who is responsible for adding a new member to respective team (ex: @open-telemetry/xxx-approvers or @open-telemetry/xxx-maintainers)?

I see that the teams allow Member vs. Maintainer roles, which exist outside of a specific repository. Do Maintainers have rights to add people to the team?

Yes: https://github.com/open-telemetry/community/blob/master/docs/how-to-configure-new-repository.md#permissions:

Even though the members of foo-maintainers are included in foo-approvers transitively, every member of foo-maintainers should be included in foo-approvers explicitly, and with the "Maintainer" GitHub privileges. So repository maintainers can invite new approvers to the team.

node teams seems to be broken. I can fix permissions right away. Also do you want them to be renamed to JavaScript?

node teams seems to be broken. I can fix permissions right away. Also do you want them to be renamed to JavaScript?

Sure, thanks 馃憤

approvers are fixed:

image

Maintainers are fixed:

image

Great to see this!

This is now done.

@lizthegrey we still have docs, erlang, cpp, and a few others not migrated out: https://github.com/open-telemetry/community/blob/master/community-members.md#docs-and-website

Let's keep it open. Would you mind following up with these SIGs?

Sure, I can sort that out.

There are some special sigs that are cross repos (like TC) I would suggest for this we keep track here.

I believe this is done for all, except website. Let's track decentralization of website sig membership file in this issue https://github.com/open-telemetry/community/issues/427

Was this page helpful?
0 / 5 - 0 ratings

Related issues

blumamir picture blumamir  路  4Comments

rakyll picture rakyll  路  4Comments

RassK picture RassK  路  3Comments

robbkidd picture robbkidd  路  3Comments

mtwo picture mtwo  路  4Comments