<div dir="ltr">Hi Marios,<div><br></div><div>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. </div>
<div><br></div><div>Chris Krelle</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Apr 21, 2014 at 9:38 AM, <a href="mailto:marios@redhat.com">marios@redhat.com</a> <span dir="ltr"><<a href="mailto:mandreou@redhat.com" target="_blank">mandreou@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I think both PTL candidates mentioned process improvements wrt<br>
contributions and reviews in their candidacy announcements. As a new<br>
Neutron dev I have seen that it is easy for reviews to go unnoticed,<br>
especially when they are stand-alone bug fixes that aren't part of a<br>
particular blueprint group (and so aren't discussed/highlighted at the<br>
various sub-team meetings). Of course this is also compounded by a<br>
seemingly heavy backlog of reviews. I realise that all core/non-core<br>
devs are doing as much as they can and though more cores will help, it<br>
takes a long time to develop people into this role.<br>
<br>
I was wondering if a 'review hour' would help. This is something Lucas<br>
Gomez told me about; the Ironic core team has a weekly hour slot in<br>
which they discuss x number of reviews and approve or -1 them. Besides<br>
getting reviews cleared quicker, it also opens the process up. It will<br>
allow new contributors to (more quickly) learn about the kinds of things<br>
core reviewers look for in a patch and also give real-time feedback<br>
(e.g. could just use #openstack-neutron for discussion during the hour).<br>
I feel that this could have an impact on the review backlog even if this<br>
only tackling the oldest 5 reviews for example.<br>
<br>
Do any of the core devs think this would be a good thing, and do you<br>
think you have the time for it?<br>
<br>
thanks, marios<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br></div>