knative / community

Knative governance and community material.
https://knative.dev/community
Other
252 stars 234 forks source link

PROCESS CHANGE: WG leads should be in CNCF's maintainers.csv #1383

Closed aliok closed 1 day ago

aliok commented 1 year ago

During https://github.com/knative/community/issues/945 I see TOC and SC members are added to the list in https://github.com/cncf/foundation/blob/main/project-maintainers.csv.

However, we should have more people there.

For example, Istio has different groups (https://github.com/cncf/foundation/blob/main/project-maintainers.csv#L1179):

We can go with the same structure.

Expected benefits

Expected costs

UPDATE: see this comment: https://github.com/knative/community/issues/1383#issuecomment-1865809475 . We'll add WG leads and not all approvers.

aliok commented 1 year ago

cc @knative/steering-committee

aliok commented 1 year ago

Some notes:

Also, @csantanapr , it was mentioned that you might have an objection for this. Any comments?

Similarly, @puerco @nainaz @salaboy @nainaz , any comments?

lance commented 1 year ago
  • @lance to lead the discussion for those criteria

I've sent a PR with some language for removal - voluntary and not - in the ROLES.md document here: https://github.com/knative/community/pull/1390. Feedback is welcome - there are a few questions and consider it a WIP until we come to agreement.

aliok commented 11 months ago

Update, in SC meeting, we had a vote and agreement for having WG leads in maintainers.csv file as well: https://docs.google.com/document/d/16Rpd2nhmLWrkFJUfpJ31Ox2PpultJ-bYIhqaiWQff_I/edit#bookmark=id.8jsaovwwr9fw

Not all approvers, but WG leads (in addition to SC+TOC members)

Updating the ticket description above.

aliok commented 10 months ago

PR https://github.com/cncf/foundation/pull/720

dprotaso commented 1 day ago

Going to close this out - I send cmierly@LF an updated list