> On 22 Jul 2015, at 09:28, Antoine Musso <hashar+...@free.fr> wrote:
> 
> Hello,
> 
> I have upgraded Zuul this morning which has a slight regression.  To be
> considered for merging, Zuul require the change to have a Verified +2.
> 
> In most case that is not a problem since the tests ran and voted V+2
> already.  But there is a few other cases where Verified is 0 or -1.
> 

This primarily affects cases where:

* Someone CR+2's a change that is so new it doesn't have the V-vote from 
jenkins-bot yet.

* Someone CR+2's a change that was previously failing but is now fixed and 
jenkins is still running the build.

So this breaks cases where someone does a provisional CR+2 with the concurrent 
dependency that Jenkins will pass.

> 
> The workaround is thus to manually vote Verified +2 in addition to
> Code-Review +2.  You might have to remove the prior CR+2 to have the
> change properly recognized.
> 
> 

Please don't (and in many repos, people can't). This bypasses Jenkins and 
causes other inconsistencies. Instead wait until Jenkins is done and then CR+2 
it.

-- Krinkle


_______________________________________________
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Reply via email to