[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:
>> 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