44928 2010-08-30 20:49:05 -0700 commit-queue gets stuck if rollout fails and tree is red 2010-08-30 20:52:26 -0700 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) PC OS X 10.5 RESOLVED DUPLICATE 37553 P2 Normal --- 44924 1 eric webkit-unassigned mihaip oldest_to_newest 271891 0 eric 2010-08-30 20:49:05 -0700 commit-queue gets stuck if rollout fails and tree is red Recent Status Patch 64903 from bug 44196: Landing patch 5 minutes ago Unable to successfully build and test [results] 5 minutes ago Patch 66003 from bug 44924: Landing rollout patch 17 minutes ago Unable to successfully build and test [results] 17 minutes ago Patch 66003 from bug 44924: Landing rollout patch 26 minutes ago Unable to successfully build and test [results] failure message: http://webkit-commit-queue.appspot.com/results/3826166 I believe this is what's happening: Normally the cq tries 3 times when landing a patch: 1. applies patch, tries to build, run tests, if successful lands, stops processing. 2. builds and tests w/o patch applied. if fails, assumes tree is busted, stops processing, sleeps 2 minutes. 3. applies patch, tries to build, run tests, if succeeds, lands. if fails, rejects patch. this allows us to deal with flaky tests better. I believe we already have a bug about svn-apply failures going through these steps when instad they should fail out early. Build errors should also fail early since they're never flaky. That would solve the case seen today where the apply failed. 271894 1 mihaip 2010-08-30 20:50:49 -0700 I think bug 37553 is about svn-apply failing. 271895 2 eric 2010-08-30 20:51:53 -0700 Yup, this will basically end up being a duplicate of bug 37553. We just need bug 37553 to handle the build case too. 271896 3 eric 2010-08-30 20:52:26 -0700 *** This bug has been marked as a duplicate of bug 37553 ***