Open Stack

Hi Marios,

Just my two cents. I have found these "Review Jam's" to be quite useful.
Ironic has been able to accomplish quite a bit having this style of review,
and I believe been able to provide good concise reviews back to the Devs.

Chris Krelle


On Mon, Apr 21, 2014 at 9:38 AM, marios at redhat.com <mandreou at redhat.com>wrote:

> Hi,
>
> I think both PTL candidates mentioned process improvements wrt
> contributions and reviews in their candidacy announcements. As a new
> Neutron dev I have seen that it is easy for reviews to go unnoticed,
> especially when they are stand-alone bug fixes that aren't part of a
> particular blueprint group (and so aren't discussed/highlighted at the
> various sub-team meetings). Of course this is also compounded by a
> seemingly heavy backlog of reviews. I realise that all core/non-core
> devs are doing as much as they can and though more cores will help, it
> takes a long time to develop people into this role.
>
> I was wondering if a 'review hour' would help. This is something Lucas
> Gomez told me about; the Ironic core team has a weekly hour slot in
> which they discuss x number of reviews and approve or -1 them. Besides
> getting reviews cleared quicker, it also opens the process up. It will
> allow new contributors to (more quickly) learn about the kinds of things
> core reviewers look for in a patch and also give real-time feedback
> (e.g. could just use #openstack-neutron for discussion during the hour).
> I feel that this could have an impact on the review backlog even if this
> only tackling the oldest 5 reviews for example.
>
> Do any of the core devs think this would be a good thing, and do you
> think you have the time for it?
>
> thanks, marios
>
> _______________________________________________
> 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/20140421/7715f92b/attachment.html>