[openstack-dev] additional core review criteria - recent Jenkins pass - otherwise you break the gates

Sean Dague sean at dague.net
Fri Dec 20 00:41:21 UTC 2013


https://review.openstack.org/#/c/51793/ is a good instance of a behavior
I've seen a lot of recently, where someone approves a patch that last
ran CI on it a month ago (the last Jenkins pass on this patch was Nov 19th).

If you come across a patch like that, as a core reviewer, please
"recheck no bug" to make sure it actually passes.

This patches unit tests don't. And what happens then is it becomes a
wrecking ball.

It fails, and gets pulled to the side, causing a reset for anything
behind it. In this case with unit tests failing that means a 20 - 30
minute delay to everything behind it. If anything in front of it fails,
zuul puts it back into rotation, because the change in front of it that
failed *might have been the problem*. Then it fails again, resets the
queue behind it. Another 20 - 30 minute delay.

If there are lots of other races in the gate, and it's a long queue, a
change like this could add *hours* of gate delay.

So please look for recent passes before +Aing anything.

	-Sean

-- 
Sean Dague
Samsung Research America
sean at dague.net / sean.dague at samsung.com
http://dague.net

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 482 bytes
Desc: OpenPGP digital signature
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20131219/1d2a9178/attachment.pgp>


More information about the OpenStack-dev mailing list