Open Stack

I think that IBM has a very interesting policy in that two IBM cores should not approve a patch posted by one of their colleagues (that is what Chris RIP used to tell me). It would be nice if the community would follow this policy.
Thanks
Gary

From: "Armando M." <armamig at gmail.com<mailto:armamig at gmail.com>>
Reply-To: OpenStack List <openstack-dev at lists.openstack.org<mailto:openstack-dev at lists.openstack.org>>
Date: Sunday, February 21, 2016 at 6:40 PM
To: OpenStack List <openstack-dev at lists.openstack.org<mailto:openstack-dev at lists.openstack.org>>
Subject: Re: [openstack-dev] [kolla] discussion about core reviewer limitations by company



On 20 February 2016 at 14:06, Kevin Benton <kevin at benton.pub<mailto:kevin at benton.pub>> wrote:

I don't think neutron has a limit. There are 4 from redhat and 3 from hp and mirantis right now. https://review.openstack.org/#/admin/groups/38,members

By the way, technically speaking some of those also only limit themselves the right to merge to their area of expertise.

On Feb 20, 2016 13:02, "Steven Dake (stdake)" <stdake at cisco.com<mailto:stdake at cisco.com>> wrote:
Neutron, the largest project in OpenStack by active committers and reviewers as measured by the governance repository teamstats tool, has a limit of 2 core reviewers per company.  They do that for a reason.  I expect Kolla will grow over time (we are about 1/4 their size in terms of contributors and reviewers).  I believe other projects follow a similar pattern besides Neutron that already have good diversity (and intend to keep it in place).

Regards
-steve


From: Gal Sagie <gal.sagie at gmail.com<mailto:gal.sagie at gmail.com>>
Reply-To: "OpenStack Development Mailing List (not for usage questions)" <openstack-dev at lists.openstack.org<mailto:openstack-dev at lists.openstack.org>>
Date: Saturday, February 20, 2016 at 10:38 AM
To: "OpenStack Development Mailing List (not for usage questions)" <openstack-dev at lists.openstack.org<mailto:openstack-dev at lists.openstack.org>>
Subject: Re: [openstack-dev] [kolla] discussion about core reviewer limitations by company

I think setting these limits is wrong, some companies have more overall representation then others.
The core reviewer job should be on a personal basis and not on a company basis, i think the PTL of each project needs
to make sure the diversity and the community voice is heard in each project and the correct path is taken even if
many (or even if all) of the cores are from the same company.
If you really want to set limits then i would go with something like 2 cores from the same company cannot +2 the same patch, but
again i am against such things personally..

Disclaimer: i am not personally involved in Kolla or know how things are running there.

On Sat, Feb 20, 2016 at 7:09 PM, Steven Dake (stdake) <stdake at cisco.com<mailto:stdake at cisco.com>> wrote:
Hey folks,

Mirantis has been developing a big footprint in the core review team, and Red Hat already has a big footprint in the core review team.  These are all good things, but I want to avoid in the future a situation in which one company has a majority of core reviewers.  Since core reviewers set policy for the project, the project could be harmed if one company has such a majority.  This is one reason why project diversity is so important and has its own special snowflake tag in the governance repository.

I'd like your thoughts on how to best handle this situation, before I trigger  a vote we can all agree on.

I was thinking of something simple like:
"1 company may not have more then 33% of core reviewers.  At the conclusion of PTL elections, the current cycle's 6 months of reviews completed will be used as a metric to select the core reviewers from that particular company if the core review team has shrunk as a result of removal of core reviewers during the cycle."

Thoughts, comments, questions, concerns, etc?

Regards,
-steve


__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe<http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe>
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev




--
Best Regards ,

The G.

__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe<http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe>
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe<http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe>
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20160221/88cf6b40/attachment.html>