Bug 77026 - run-perf-tests should timeout sooner
Summary: run-perf-tests should timeout sooner
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
URL:
Keywords:
Depends on:
Blocks: 77037
  Show dependency tree / graph
 
Reported: 2012-01-25 10:30 PST by Ryosuke Niwa
Modified: 2012-01-25 12:29 PST (History)
5 users (show)

See Also:

Attachments
Timeout in 4min instead of 10m; also use float instead of string in test result values (6.33 KB, patch)
2012-01-25 10:41 PST, Ryosuke Niwa 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2012-01-25 10:30:37 PST 
It seems like a whole bunch of inspector tests are timing out on the perf bot and causing the cycle time to be over 1h :(

e.g. http://build.webkit.org/builders/Chromium%20Mac%20Release%20%28Perf%29/builds/6/steps/perf-test/logs/stdio

Given recent runs, 4 minutes (instead of 10 minutes as is) per test should be sufficient.
Comment 1 Ryosuke Niwa 2012-01-25 10:31:58 PST 
Also see https://bugs.webkit.org/show_bug.cgi?id=77024 (Web Inspector: tests in PerformanceTests/inspector/ are timing out)
Comment 2 Ryosuke Niwa 2012-01-25 10:41:29 PST 
Created attachment 123971 [details]
Timeout in 4min instead of 10m; also use float instead of string in test result values
Comment 3 Ryosuke Niwa 2012-01-25 12:29:21 PST 
Comment on attachment 123971 [details]
Timeout in 4min instead of 10m; also use float instead of string in test result values

Clearing flags on attachment: 123971

Committed r105915: <http://trac.webkit.org/changeset/105915>
Comment 4 Ryosuke Niwa 2012-01-25 12:29:24 PST 
All reviewed patches have been landed.  Closing bug.