Bug 56801 - REGRESSION(r81597): Change results in lots of unexpected flaky on the dashboard. (Requested by pfeldman on #webkit).
Summary: REGRESSION(r81597): Change results in lots of unexpected flaky on the dashboa...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: WebKit Review Bot
URL:
Keywords:
Depends on:
Blocks: 56760
  Show dependency treegraph
 
Reported: 2011-03-21 22:17 PDT by WebKit Review Bot
Modified: 2011-03-21 22:30 PDT (History)
5 users (show)

See Also:


Attachments
ROLLOUT of r81597 (4.25 KB, patch)
2011-03-21 22:18 PDT, WebKit Review Bot
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description WebKit Review Bot 2011-03-21 22:17:42 PDT
http://trac.webkit.org/changeset/81597 broke the build:
Change results in lots of unexpected flaky on the dashboard. (Requested by pfeldman on #webkit).

This is an automatic bug report generated by the sheriff-bot. If this bug
report was created because of a flaky test, please file a bug for the flaky
test (if we don't already have one on file) and dup this bug against that bug
so that we can track how often these flaky tests case pain.

"Only you can prevent forest fires." -- Smokey the Bear
Comment 1 WebKit Review Bot 2011-03-21 22:18:12 PDT
Created attachment 86420 [details]
ROLLOUT of r81597

Any committer can land this patch automatically by marking it commit-queue+.  The commit-queue will build and test the patch before landing to ensure that the rollout will be successful.  This process takes approximately 15 minutes.

If you would like to land the rollout faster, you can use the following command:

  webkit-patch land-attachment ATTACHMENT_ID --ignore-builders

where ATTACHMENT_ID is the ID of this attachment.
Comment 2 Pavel Feldman 2011-03-21 22:21:43 PDT
Comment on attachment 86420 [details]
ROLLOUT of r81597

Clearing flags on attachment: 86420

Committed r81645: <http://trac.webkit.org/changeset/81645>
Comment 3 Pavel Feldman 2011-03-21 22:21:50 PDT
All reviewed patches have been landed.  Closing bug.
Comment 4 Dirk Pranke 2011-03-21 22:27:30 PDT
If this test made a bunch of tests become suddenly flaky, then that means that they were sensitive to the order they were being run in. I think the right thing to do is probably to mark them as flaky in the expectations and then fix them, rather than roll this out.

(Of course, you've already rolled this out, so I suppose I mean that we should re-land this and then triage and mark as flaky ...).

There isn't really any good way to figure out what the flakiness will be without landing this and getting some runs in.

Thoughts?
Comment 5 Dirk Pranke 2011-03-21 22:30:05 PDT
Per comment on bug 56760, Mihai suggests that this change actually made the http tests run backwards on the chromium linux bots. That would be an actual bad thing deserving of roll-out, rather than just flakiness.