<p dir="ltr">I've always done it manually by eyeballing the review, but the script is tempting.</p>
<p dir="ltr">Thanks,<br>
Michael</p>
<div class="gmail_quote">On 27 May 2016 8:42 PM, "Sean Dague" <<a href="mailto:sean@dague.net">sean@dague.net</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 05/27/2016 05:36 AM, Michael Still wrote:<br>
> Hi,<br>
><br>
> I've spent some time today abandoning old reviews from the Nova queue.<br>
> Specifically, anything which hadn't been updated before February this<br>
> year has been abandoned with a message like this:<br>
><br>
> "This patch has been idle for a long time, so I am abandoning it to keep<br>
> the review clean sane. If you're interested in still working on this<br>
> patch, then please unabandon it and upload a new patchset."<br>
><br>
> Why do this? Abandoning the reviews means that Nova reviewers can focus<br>
> on things where the author is still actively working on the code.<br>
> Additionally, it makes it clearer which bugs are currently being worked.<br>
><br>
> Additionally, unabandoning a review is a fairly cheap operation, so<br>
> please let me know if I need to do that anywhere.<br>
><br>
> We should probably abandon more patches than those before February, but<br>
> I got bored at this point. I'll probably abandon more later.<br>
><br>
> Cheers,<br>
> Michael<br>
<br>
We have a script in tree that can be run by any core team member -<br>
<a href="https://github.com/openstack/nova/blob/c69afd454b41e2e8fc3496ff56b986342f547064/tools/abandon_old_reviews.sh#L2" rel="noreferrer" target="_blank">https://github.com/openstack/nova/blob/c69afd454b41e2e8fc3496ff56b986342f547064/tools/abandon_old_reviews.sh#L2</a><br>
<br>
<br>
It tries to describe the policy, which is basically things with no<br>
activity in the last 4 weeks, and has a -2 or a Jenkins -1 on it.<br>
<br>
The biggest issue here is the procedural -2s that don't tend to lift<br>
right away after release (which is probably a mistake, we should only<br>
really use procedural -2s during freeze windows). Feel free to modify<br>
accordingly.<br>
<br>
        -Sean<br>
<br>
--<br>
Sean Dague<br>
<a href="http://dague.net" rel="noreferrer" target="_blank">http://dague.net</a><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div>