[openstack-dev] [nova] A modest proposal to reduce reviewer load

Russell Bryant rbryant at redhat.com
Tue Jun 17 12:32:21 UTC 2014


On 06/17/2014 08:20 AM, Daniel P. Berrange wrote:
> On Tue, Jun 17, 2014 at 01:12:45PM +0100, Matthew Booth wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> On 17/06/14 12:36, Sean Dague wrote:
>>> On 06/17/2014 07:23 AM, Daniel P. Berrange wrote:
>>>> If would be nice if gerrit had simple keyword tagging so any
>>>> reviewer can tag an existing commit as "trivial", but that
>>>> doesn't seem to exist as a concept yet.
>>>>
>>>> So an alternative perhaps submit trivial stuff using a well
>>>> known topic eg
>>>>
>>>> # git review --topic trivial
>>>>
>>>> Then you can just query all changes in that topic to find easy
>>>> stuff to approve.
>>>
>>> It could go in the commit message:
>>>
>>> TrivialFix
>>>
>>> Then could be queried with - 
>>> https://review.openstack.org/#/q/message:TrivialFix,n,z
>>>
>>> If a reviewer felt it wasn't a trivial fix, they could just edit
>>> the commit message inline to drop it out.
> 
> Yes, that would be a workable idea.
> 
>> +1. If possible I'd update the query to filter out anything with a -1.
>>
>> Where do we document these things? I'd be happy to propose a docs update.
> 
> Lets see if any other nova cores dissent, but then can add it to these 2
> wiki pages
> 
>   https://wiki.openstack.org/wiki/ReviewChecklist
>   https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references

Seems reasonable to me.

Of course, I just hope it doesn't put reviewers in a mode of only
looking for the trivial stuff and helping less with the big stuff.

-- 
Russell Bryant



More information about the OpenStack-dev mailing list