[openstack-dev] [Nova] Spring cleaning nova-core

Gary Kotton gkotton at vmware.com
Sun Dec 7 10:48:47 UTC 2014


Please see http://stackalytics.com/report/contribution/nova-group/90. If
we are following the average of 2 reviews per day then proposed list
should be updated.

On 12/7/14, 11:19 AM, "Michael Still" <mikal at stillhq.com> wrote:

>On Sun, Dec 7, 2014 at 7:03 PM, Gary Kotton <gkotton at vmware.com> wrote:
>> On 12/6/14, 7:42 PM, "Jay Pipes" <jaypipes at gmail.com> wrote:
>
>[snip]
>
>>>-1 on pixelbeat, since he's been active in reviews on
>>>various things AFAICT in the last 60-90 days and seems to be still a
>>>considerate reviewer in various areas.
>>
>> I agree -1 for Padraig
>
>I'm going to be honest and say I'm confused here.
>
>We've always said we expect cores to maintain an average of two
>reviews per day. That's not new, nor a rule created by me. Padraig is
>a great guy, but has been working on other things -- he's done 60
>reviews in the last 60 days -- which is about half of what we expect
>from a core.
>
>Are we talking about removing the two reviews a day requirement? If
>so, how do we balance that with the widespread complaints that core
>isn't keeping up with its workload? We could add more people to core,
>but there is also a maximum practical size to the group if we're going
>to keep everyone on the same page, especially when the less active
>cores don't generally turn up to our IRC meetings and are therefore
>more "expensive" to keep up to date.
>
>How can we say we are doing our best to keep up with the incoming
>review workload if all reviewers aren't doing at least the minimum
>level of reviews?
>
>Michael
>
>-- 
>Rackspace Australia
>
>_______________________________________________
>OpenStack-dev mailing list
>OpenStack-dev at lists.openstack.org
>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev




More information about the OpenStack-dev mailing list