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

Sync the Notary maintainers to CNCF maintainer list #3

Closed
FeynmanZhou opened this issue Feb 28, 2023 · 18 comments
Closed

Sync the Notary maintainers to CNCF maintainer list #3

FeynmanZhou opened this issue Feb 28, 2023 · 18 comments
Labels
Governance issues to improve governance posture

Comments

@FeynmanZhou
Copy link
Member

FeynmanZhou commented Feb 28, 2023

CNCF uses the maintainer list as a reference to grant permissions to the project maintainers in some CNCF events, systems, or programs. However, there are only Notary Org owners listed on the CNCF maintainer list. To add other Notary owners to the CNCF maintainer list, we need to sync the Notary maintainers from MAINTAINERS to the CNCF maintainer list.

@toddysm @vaninrao10 @priteshbandi @yizha1 @iamsamirzon
Do you have any concerns or suggestions about this proposal?

@toddysm
Copy link
Contributor

toddysm commented Mar 4, 2023

LGTM

2 similar comments
@yizha1
Copy link
Contributor

yizha1 commented Mar 4, 2023

LGTM

@priteshbandi
Copy link

LGTM

@SteveLasker
Copy link
Contributor

As mentioned earlier, the ask from CNCF (via @mattfarina) wasn't to surface every maintainer from each repo. Rather, the org maintainers as not every maintainer needs access to the CNCF tooling. We did discuss adding new org maintainers to provide more folks access.

@toddysm
Copy link
Contributor

toddysm commented Mar 7, 2023

@SteveLasker @FeynmanZhou linked the MAINTAINERS file for this (.github) repository that lists a subset of the list of maintainers on all the sub-projects. It does make sense to have the people who drive the governance to be the folks CNCF communicates with. It would be good to have this list of people also as Org maintainers but this is up to the current Org maintainers to decide on.

@SteveLasker
Copy link
Contributor

  • We have org maintainers - which we should expand
  • We have sub-project maintainers, which have expanded - great.
  • I'm hearing you define a new group, (> org maintainers, but < sub-project maintainers)

The org maintainers are the ones that should be working on the org governance, dealing with CNCF tools and such. Rather than define yet another group, I'd suggest we nominate additional org maintainers that can be active in the day-to-day operations.

@iamsamirzon
Copy link
Contributor

I like the idea of expanding the list of org maintainers to include the people active in the day-to-day operations and setting the standards.

@vaninrao10
Copy link

LGTM - expanding the list of org maintainers to include the people active in the day-to-day operations and setting the standards.

@toddysm
Copy link
Contributor

toddysm commented Mar 8, 2023

@justincormack @SteveLasker @NiazFK Do we have your approval to expand the list of Org Maintainers with the folks listed in MAINTAINERS?

@justincormack
Copy link

SGTM

@NiazFK
Copy link

NiazFK commented Mar 14, 2023

LGTM

@SteveLasker
Copy link
Contributor

Yes to adding org maintainers as that has been the goal for a while. The only caveat discussed was keeping the org maintainers balanced across orgs, avoiding any one company from becoming a majority. @NiazFK representing AWS, @justincormack representing Docker and @toddysm representing Microsoft should nominate folks. We agreed to keep this at 3 per company.

@toddysm
Copy link
Contributor

toddysm commented Mar 15, 2023

Based on the current list in the MAINTAINERS the balance is: 4 AWS/3 MSFT/1 Docker/1 Independent. There is no simple majority for one company, and it will be hard to get super-majority without diverse votes. It would be good to get more folks from Docker to make it more balanced but I am OK with using the current list so we can unblock some work that is pending for a while already.

I will bring this to the next Notary meeting (3/20) for final agreement.

@SteveLasker
Copy link
Contributor

I'd suggest closing this issue and open new issues for individual maintainer nominations to have proper tracking of org maintainer changes.

@toddysm
Copy link
Contributor

toddysm commented Mar 16, 2023

I believe we have enough approvals on this issue to use the current MAINTAINERS list to sync as Org maintainers. We can always nominate new ones and amend the current list. What is the purpose of having a new one and starting over? We have been discussing this for a while already and it is blocking the work of the folks who are actively working on the project.

@toddysm
Copy link
Contributor

toddysm commented Mar 21, 2023

As per the agreement from the Notary meeting today, the next steps we will take on this are the following:

@FeynmanZhou
Copy link
Member Author

We can close this issue as cncf/foundation#524 has been merged.

@FeynmanZhou
Copy link
Member Author

@toddysm I sent an email to [email protected] but it seems this email is invalid. I will ping Amye for this problem.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Governance issues to improve governance posture
Projects
Status: Done
Development

No branches or pull requests

9 participants