On 22 Jul 2015, at 09:28, Antoine Musso hashar+wmf@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