Bug 144097

Summary: [buildbot] Fix the output of perf-test buildstep on timeout
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: New BugsAssignee: Csaba Osztrogonác <ossy>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, lforschler, ossy, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

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.