Bug 144097 - [buildbot] Fix the output of perf-test buildstep on timeout
Summary: [buildbot] Fix the output of perf-test buildstep on timeout
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Csaba Osztrogonác
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-23 05:59 PDT by Csaba Osztrogonác
Modified: 2015-04-27 03:06 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.36 KB, patch)
2015-04-23 06:23 PDT, Csaba Osztrogonác
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2015-04-23 05:59:44 PDT
https://build.webkit.org/builders/EFL%20Linux%2064-bit%20Release%20WK2%20%28Perf%29/builds/5423
result:  -1 perf tests failed

stdout:
...
command timed out: 1200 seconds without output running ['python', './Tools/Scripts/run-perf-tests', '--output-json-path', 'perf-test-results.json', '--slave-config-json-path', '../../perf-test-config.json', '--no-show-results', '--reset-results', '--test-results-server', 'perf.webkit.org', '--builder-name', 'EFL Linux 64-bit Release WK2 (Perf)', '--build-number', '5423', '--platform', 'efl', '--no-build', '--release'], attempting to kill
process killed by signal 9
program finished with exit code -1
elapsedTime=2612.764758

----

Now only 255, 254, 253, 252 and 251 are handled explicitly,
otherwise the return code means the number of failing tests.

But now buildbot killed the script after 20 minutes timeout and 
set the return code to -1. (From the script we can get 0-255)

We should handle this -1 explicitly.
Comment 1 Csaba Osztrogonác 2015-04-23 06:23:55 PDT
Created attachment 251428 [details]
Patch
Comment 2 Csaba Osztrogonác 2015-04-27 03:06:28 PDT
Comment on attachment 251428 [details]
Patch

Clearing flags on attachment: 251428

Committed r183390: <http://trac.webkit.org/changeset/183390>
Comment 3 Csaba Osztrogonác 2015-04-27 03:06:36 PDT
All reviewed patches have been landed.  Closing bug.