openstack team mailing list archive
-
openstack team
-
Mailing list archive
-
Message #13052
Re: [Netstack] [Quantum] Review days for Quantum core devs
As the link to the wiki page is now buried deep in this thread, here it is
again: http://wiki.openstack.org/Quantum/ReviewDays
Weekend days have been added on GaryK's request!
Salvatore
On 11 June 2012 16:51, Dan Wendlandt <dan@xxxxxxxxxx> wrote:
>
>
> On Mon, Jun 11, 2012 at 2:25 AM, Salvatore Orlando <salv.orlando@xxxxxxxxx
> > wrote:
>
>> Hi,
>>
>> During our last project meeting there was some interest in introducing
>> "review days" for Quantum core devs.
>> I see multiple benefits in this approach:
>>
>> 1) Contributors would know which core member the need to contact on a
>> given days for soliciting a review for their patches
>> 2) Predictable code review effort for core devs
>> 3) The PTL knows who should be whipped!
>>
>
> indeed :)
>
> Thanks for taking the lead on this Salvatore. Please send out the wiki
> page, no need for more +1s :)
>
> dan
>
>
>>
>> I think we can leverage the fact that our core devs span several time
>> zones. So for each day we can have, for instance, an EMEA core reviewer on
>> duty between 1AM and 9AM PST, an US core reviewer on duty between 9AM and
>> 5PM PST, and an APAC core reviewer on duty between 5PM and 1AM PST.
>> Of course the plan needs to be flexible especially around milestone
>> release, where all the devs which are not busy coding should probably
>> dedicate more time to code review.
>>
>> I am going to start a wiki page with a 5x3 grid, with days on the rows
>> and zones on the columns. If you like this idea, +1 this email and add your
>> name to the grid :)
>> Otherwise, please let me have your feedback and concerns.
>>
>> Salvatore
>>
>> --
>> Mailing list: https://launchpad.net/~netstack
>> Post to : netstack@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~netstack
>> More help : https://help.launchpad.net/ListHelp
>>
>>
>
>
> --
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~
> Dan Wendlandt
> Nicira, Inc: www.nicira.com
> twitter: danwendlandt
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
>
Follow ups
References