Bug 61441 - TestFailures page should show when run-webkit-tests started exiting early due to too many crashes, timeouts, or failures
Summary: TestFailures page should show when run-webkit-tests started exiting early due...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL: http://build.webkit.org/TestFailures
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-25 08:47 PDT by Adam Roben (:aroben)
Modified: 2011-05-25 10:39 PDT (History)
2 users (show)

See Also:


Attachments
Identify, rather than skip, builds where run-webkit-tests exited early due to too many failures (13.35 KB, patch)
2011-05-25 09:28 PDT, Adam Roben (:aroben)
ddkilzer: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 2011-05-25 08:47:02 PDT
If enough tests crash, time out, or fail, run-webkit-tests will exit early to avoid slowing down the bots. It would be good for the TestFailures page to be able to tell you when this started happening so that the guilty revision can be identified.
Comment 1 Adam Roben (:aroben) 2011-05-25 08:47:50 PDT
Right now the TestFailures page doesn't even list testers for which this is happening.
Comment 2 Adam Roben (:aroben) 2011-05-25 09:28:06 PDT
Created attachment 94793 [details]
Identify, rather than skip, builds where run-webkit-tests exited early due to too many failures
Comment 3 David Kilzer (:ddkilzer) 2011-05-25 09:36:45 PDT
Comment on attachment 94793 [details]
Identify, rather than skip, builds where run-webkit-tests exited early due to too many failures

r=me
Comment 4 Adam Roben (:aroben) 2011-05-25 09:44:37 PDT
Committed r87299: <http://trac.webkit.org/changeset/87299>
Comment 5 Joseph Pecoraro 2011-05-25 10:39:43 PDT
Comment on attachment 94793 [details]
Identify, rather than skip, builds where run-webkit-tests exited early due to too many failures

View in context: https://bugs.webkit.org/attachment.cgi?id=94793&action=review

> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Builder.js:140
>       * Preiodically calls callback until all current failures have been explained. Callback is

Typo in this comment unchanged by your patch: "Preiodically" => "Periodically"