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

Erlon Cruz sombrafam at gmail.com
Wed Jun 18 19:48:10 UTC 2014


Great John,

I'll push fellows here to help. Couldn't  join today but I'll put in my
schedule for the next weeks. Do you plan to do this review force every
Wednesday?

Erlon


On Mon, Jun 16, 2014 at 9:31 AM, Kerr, Andrew <Andrew.Kerr at netapp.com>
wrote:

> +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
>
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> 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/20140618/99243f98/attachment-0001.html>


More information about the OpenStack-dev mailing list