[openstack-dev] [OpenStack-Dev][Cinder] Review days? (open to ANYBODY and EVERYBODY)

Kerr, Andrew Andrew.Kerr at netapp.com
Mon Jun 16 12:31:31 UTC 2014


+1

Andrew Kerr


On 6/13/14, 10:30 AM, "Duncan Thomas" <duncan.thomas at gmail.com> wrote:

>Same as Jay, for much the same reasons. Having a fixed calendar time
>makes it easy for me to put up a 'do not disturb' sign.
>
>On 13 June 2014 05:10, Jay Bryant <jsbryant at electronicjungle.net> wrote:
>> John,
>>
>> +2
>>
>> I am guilty of falling behind on reviews. Pulled in to a lot of other
>>stuff
>> since the summit ... and before.
>>
>> Having prescribed time on my calendar is a good idea.  Just put it on my
>> calendar.
>>
>> Jay
>>
>> On Jun 12, 2014 10:49 PM, "John Griffith" <john.griffith at solidfire.com>
>> wrote:
>>>
>>> Hey Everyone,
>>>
>>> So I've been noticing some issues with regards to reviews in Cinder
>>> lately, namely we're not keeping up very well.  Most of this is a math
>>> problem (submitters >> reviewers).  We're up around 200+ patches in the
>>> queue, and a large number of them have no negative feedback but have
>>>just
>>> been waiting patiently (some > 2 months).
>>>
>>> Growth is good, new contributors are FANTASTIC... but stale
>>>submissions in
>>> the queue are BAD, and I hate for people interested in contributing to
>>> become discouraged and just go away (almost as much as I hate emails
>>>asking
>>> me to review patches).
>>>
>>> I'd like to propose we consider one or two review days a week for a
>>>while
>>> to try and work on our backlog.  I'd like to propose that on these
>>>days we
>>> make an attempt to NOT propose new code (or at least limit it to
>>>bug-fixes
>>> [real bugs, not features disguised as bugs]) and have an agreement from
>>> folks to focus on actually doing reviews and using IRC to collaborate
>>> together and knock some of these out.
>>>
>>> We did this sort of thing over a virtual meetup and it was really
>>> effective, I'd like to see if we can't do something for a brief
>>>duration
>>> over IRC.
>>>
>>> I'm thinking we give it a test run, set aside a few hours next Wed
>>>morning
>>> to start (coinciding with our Cinder weekly meeting since many folks
>>>around
>>> that morning across TZ's etc) where we all dedicate some time prior to
>>>the
>>> meeting to focus exclusively on helping each other get some reviews
>>>knocked
>>> out.  As a reminder Cinder weekly meeting is 16:00 UTC.
>>>
>>> Let me know what you all think, and keep in mind this is NOT limited to
>>> just current regular "Block-Heads" but anybody in the OpenStack
>>>community
>>> that's willing to help out and of course new reviewers are MORE than
>>> welcome.
>>>
>>> Thanks,
>>> John
>>>
>>> _______________________________________________
>>> OpenStack-dev mailing list
>>> OpenStack-dev at lists.openstack.org
>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>>
>>
>> _______________________________________________
>> OpenStack-dev mailing list
>> OpenStack-dev at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>
>
>
>
>-- 
>Duncan Thomas
>
>_______________________________________________
>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