Open Stack

Gerrit 2.8 allows setting label values on patch sets either thru the
command line[1] or REST API[2].  Since we will setup WIP as a -1 score
on a label this will just be a matter of updating git-review to set
the label on new patchsets.  I'm no sure if there's a bug entered in
our the issue tracker for this but you are welcome to create one.

[1] https://review-dev.openstack.org/Documentation/cmd-review.html
[2] https://review-dev.openstack.org/Documentation/rest-api-changes.html#set-review


On Fri, Apr 25, 2014 at 1:56 AM, Ricardo Carrillo Cruz
<ricardo.carrillo.cruz at gmail.com> wrote:
> It would be nice to have a switch on git-review to push patchsets on WIP,
> the same way you can do with Draft.
>
> Anyone knows if there is an enhancement request/bug opened against Gerrit so
> we could implement it?
>
> Regards
>
>
> 2014-04-25 10:49 GMT+02:00 Roman Podoliaka <rpodolyaka at mirantis.com>:
>
>> Hi all,
>>
>> >>> Wouldn't it be better to make this label more persistent?
>>
>> +1. It's really annoying to press "Work in Progress" button every time
>> you upload a new patch set.
>>
>> Thanks,
>> Roman
>>
>> On Fri, Apr 25, 2014 at 11:02 AM, Yuriy Taraday <yorik.sar at gmail.com>
>> wrote:
>> > Hello.
>> >
>> > On Wed, Apr 23, 2014 at 2:40 AM, James E. Blair <jeblair at openstack.org>
>> > wrote:
>> >>
>> >> * The new "Workflow" label will have a "-1 Work In Progress" value
>> >> which
>> >>   will replace the "Work In Progress" button and review state.  Core
>> >>   reviewers and change owners will have permission to set that value
>> >>   (which will be removed when a new patchset is uploaded).
>> >
>> >
>> > Wouldn't it be better to make this label more persistent?
>> > As I remember there were some ML threads about keeping WIP mark across
>> > patch
>> > sets. There were even talks about changing git-review to support this.
>> > How about we make it better with the new version of Gerrit?
>> >
>> > --
>> >
>> > Kind regards, Yuriy.
>> >
>> > _______________________________________________
>> > OpenStack-dev mailing list
>> > OpenStack-dev at lists.openstack.org
>> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>> >
>>
>> _______________________________________________
>> OpenStack-dev mailing list
>> OpenStack-dev at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>