New workflow flag on gerrit jenkins project

David Caro dcaroest at redhat.com
Thu Apr 16 19:23:08 UTC 2015


On 04/15, Eyal Edri wrote:
> +1. Excellent.
> Will it be possible to tell gerrit trigger plugin to run jobs only on patches with this flag?
> 
> If not, might worth sending a patch to support it. It might reduce dramatically the amount of jobs running on patches.

Yes, the gerrit plugin does allow filtering by any custom flag (just checked)

> 
> Sent from Nine
> 
> From: David Caro
> Sent: Apr 15, 2015 7:31 PM
> To: infra at ovirt.org
> Subject: New workflow flag on gerrit jenkins project
> 
> 
> Hi!
> 
> 
> As a test to improve the review process I've enabled a new flag on the jenkins
> repo, the workflow flag.
> 
> This flag has three values:
>   0  Work in progress (default)
>   +1 Ready for review
>   +2 Ready for merge
> 
> The idea is to work on a patch without bothering anyone and when ready,
> explicitly ask for review in gerrit, that way the reviewer can easily
> differenciate between ready and not ready patches. Once the reviews have been
> done and all, the maintainer can set the +2 and merge the patch (the idea is in
> the near future use that flag to trigger zuul, so replacing the submit button).
> 
> The new flag shows nicely in the changes screen, making in really easy to see
> which patches require reviews.
> 
> Let's play on that project only for now and see if it works out.
> 
> 
> Thanks!!
> 
> 
> -- 
> David Caro
> 
> Red Hat S.L.
> Continuous Integration Engineer - EMEA ENG Virtualization R&D
> 
> Tel.: +420 532 294 605
> Email: dcaro at redhat.com
> Web: www.redhat.com
> RHT Global #: 82-62605

-- 
David Caro

Red Hat S.L.
Continuous Integration Engineer - EMEA ENG Virtualization R&D

Tel.: +420 532 294 605
Email: dcaro at redhat.com
Web: www.redhat.com
RHT Global #: 82-62605
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://lists.ovirt.org/pipermail/infra/attachments/20150416/af7c31c9/attachment.sig>


More information about the Infra mailing list