[openstack-dev] Fixing the Nova Core Reviewer Frustration [was Re: [Nova] PTL Candidacy]
chris.friesen at windriver.com
Tue Apr 7 20:24:19 UTC 2015
On 04/07/2015 01:35 PM, James Bottomley wrote:
> If I look at the history, I also see some reviewers dropping out once
> their concerns and review comments have been addressed (after giving a
> +1), so the other thing I'd suggest is that instead of erasing the
> review history on each patch submission, it carries over (at least the
> -1 and +1) so you don't have to wait a while for a consensus to form
> (reviewers would, of course, be able to alter their votes at any time).
> The pressure is thus on the submitter to make the changes to switch
> every original -1 to a +1.
How would we deal with the scenario where someone leaves a -1 with a comment and
then never goes back to check if their concerns were dealt with?
More information about the OpenStack-dev