But I don't think I gave the topic a +2. Am I missing something?<br><br><div class="gmail_quote">On Tue, Sep 11, 2012 at 7:05 PM, Marcus D. Hanwell <span dir="ltr"><<a href="mailto:marcus.hanwell@kitware.com" target="_blank">marcus.hanwell@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">That is what I was saying - the change reviews do not matter from a<br>
merge point of view. A -1 would never block anyway (-2 required to<br>
block). Currently change scores are not able to block topic merges, so<br>
even a -2 would not have blocked because it is all on the topic review<br>
right now.<br>
<br>
By giving a +2 to the topic it satisfied all requirements (the topic<br>
score is the only score assessed by Gerrit when deciding if a change<br>
can be merged or not, robot already gave it +1 Verified).<br>
<br>
<a href="http://review.source.kitware.com/#/t/1250/" target="_blank">http://review.source.kitware.com/#/t/1250/</a><br>
<br>
In future I think any blocker in a change should block a topic from<br>
being merged, but we have not had time to implement that feature.<br>
<span class="HOEnZb"><font color="#888888"><br>
Marcus<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Tue, Sep 11, 2012 at 6:55 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>
> So here I had a -1: <a href="http://review.source.kitware.com/#/c/7465/" target="_blank">http://review.source.kitware.com/#/c/7465/</a><br>
> How did I merge it?<br>
><br>
><br>
><br>
> On Tue, Sep 11, 2012 at 6:52 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>><br>
> wrote:<br>
>><br>
>> I thought I had a -1 on a change or review. And somehow I merged it.<br>
>> Probably a screwup on my part.<br>
>><br>
>><br>
>> On Tue, Sep 11, 2012 at 6:45 PM, Marcus D. Hanwell<br>
>> <<a href="mailto:marcus.hanwell@kitware.com">marcus.hanwell@kitware.com</a>> wrote:<br>
>>><br>
>>> Sorry if I made them sound too complicated, they are quite simple.<br>
>>> Only the topic level review affects whether a topic can be merged or<br>
>>> not. In order to qualify for merging a topic must have +1 Verified<br>
>>> (set by a robot for VTK) and +2 Code Review (set by a reviewer).<br>
>>> Without them it cannot be merged.<br>
>>><br>
>>> It gets a little more complex - if you feel a patch is bad you can<br>
>>> block it. A score of -1 Verified (set by robot), or -2 Code Review<br>
>>> (set by reviewer) will prevent a merge until that score is changed or<br>
>>> removed, i.e. even if I give a topic +2 review, if it has a -2 review<br>
>>> I cannot merge it  until that score is changed or removed.<br>
>>><br>
>>> I hope that is clearer. We would like to link blockers at the<br>
>>> individual change level to the topic, but currently the change reviews<br>
>>> are summarized in the topic view but are not enforced as rules.<br>
>>><br>
>>> Marcus<br>
>>><br>
>>> On Tue, Sep 11, 2012 at 6:18 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>><br>
>>> wrote:<br>
>>> > The rules for a merge sound complicated to me.<br>
>>> ><br>
>>> > But, I will be very careful in the future.<br>
>>> ><br>
>>> ><br>
>>> > On Tue, Sep 11, 2012 at 6:10 PM, Marcus D. Hanwell<br>
>>> > <<a href="mailto:marcus.hanwell@kitware.com">marcus.hanwell@kitware.com</a>> wrote:<br>
>>> >><br>
>>> >> On Tue, Sep 11, 2012 at 5:39 PM, Sean McBride<br>
>>> >> <<a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a>><br>
>>> >> wrote:<br>
>>> >> > On Tue, 11 Sep 2012 17:19:12 -0400, Bill Lorensen said:<br>
>>> >> ><br>
>>> >> >>I hit the wrong button by accident. I did not give it a +2. I'm<br>
>>> >> >> pretty<br>
>>> >> >> sure<br>
>>> >> >>I gave it (or a change) a -1.<br>
>>> >> ><br>
>>> >> > IIRC, the topic had several changes, and some of the changes I think<br>
>>> >> > were given +2.<br>
>>> >> ><br>
>>> >> > Is it possible that the whole topic can get merged if any one of,<br>
>>> >> > but<br>
>>> >> > not all, of the sub-changes are at +2?<br>
>>> >> ><br>
>>> >> For Gerrit topic review it does not matter what any of the individual<br>
>>> >> changes get as a review. The entire topic must have a Verified +1, and<br>
>>> >> a Code Review +2 score in order to be merged. Reviewers can block a<br>
>>> >> merge by giving a -2 at the topic level, that topic had a single +2<br>
>>> >> given by one user and no other scores.<br>
>>> >><br>
>>> >> Any -1 score does not block a commit, any +1 does not enable the<br>
>>> >> submit and merge or submit change button. If you review a commit and<br>
>>> >> it does not have a +1 Verified and +2 Code Review you will see an<br>
>>> >> error page informing you the change needs Verified and Code Review. I<br>
>>> >> tested this myself on a change earlier today, it blocks merging until<br>
>>> >> both those scores are in.<br>
>>> >><br>
>>> >> I can't see any way that an accidental merge can be done in less than<br>
>>> >> two steps (+2 Code Review score, and then hitting Publish and Submit).<br>
>>> >> It is possible there is a bug, but I was not able to trigger it (and I<br>
>>> >> tried a few times).<br>
>>> >><br>
>>> >> Marcus<br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> ><br>
>>> > --<br>
>>> > Unpaid intern in BillsBasement at noware dot com<br>
>>> ><br>
>><br>
>><br>
>><br>
>><br>
>> --<br>
>> Unpaid intern in BillsBasement at noware dot com<br>
>><br>
><br>
><br>
><br>
> --<br>
> Unpaid intern in BillsBasement at noware dot com<br>
><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Unpaid intern in BillsBasement at noware dot com<br><br>