<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 8, 2016 at 1:42 PM, Dmitry Tantsur <span dir="ltr"><<a href="mailto:divius.inside@gmail.com" target="_blank">divius.inside@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><span class="">2016-04-08 19:26 GMT+02:00 Davanum Srinivas <span dir="ltr"><<a href="mailto:davanum@gmail.com" target="_blank">davanum@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Team,<br>
<br>
Steve pointed out to a problem in Stackalytics:<br>
<a href="https://twitter.com/stevebot/status/718185667709267969" rel="noreferrer" target="_blank">https://twitter.com/stevebot/status/718185667709267969</a></blockquote><div><br></div></span><div>There are many ways to game a simple +1 counter, such as +1'ing changes that already have at least 1x +2, or which already approved, or which need rechecking...<br></div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
It's pretty clear what's happening if you look here:<br>
<a href="https://review.openstack.org/#/q/owner:openstack-infra%2540lists.openstack.org+status:open" rel="noreferrer" target="_blank">https://review.openstack.org/#/q/owner:openstack-infra%2540lists.openstack.org+status:open</a><br>
<br>
Here's the drastic step (i'd like to avoid):<br>
<a href="https://review.openstack.org/#/c/303545/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/303545/</a><br>
<br>
What do you think?<br></blockquote><div><br></div></span><div>One more possible (though also imperfect) mitigation is to make exception from the usual 2x +2 rule for requirements updates passing gates and use only 1x +2. Then requirements reviews will take substantially less time to land, reducing need/possibility of having such +1's.<br></div><span class=""><div> </div></span></div></div></div></blockquote><div><br></div><div>Many projects (including Keystone) already go with a single core +2/+A for Proposal Bot changes. This definitely helps them land faster (which is why the policy is there).  </div></div></div></div>