417260 - Sisyphus/JavaScript Tests - test timeout handling

Status: RESOLVED FIXED

(Testing Graveyard :: Sisyphus, defect)

Description

[Bob Clary [:bc] (inactive)]

Attachment: patch

Due to the use of VMs the windows browser tests are sometimes taking more time than the current limit of 3 hours. This patch increases the time out to 5 hours for the browser tests. 

This patch also includes additional adjustments to public-failures.txt to include additional tests which time out. This will exclude these tests from normal runs and speed up the test runs.

Rob, it appears there is also a 1 hour timeout on the js tests imposed by buildbot and an 8 hour timeout on the browser tests. I think the browser timeout is fine, but can we increase the shell timeout on all slaves to be 3 hours as well?

Comment 1

[Rob Campbell [:rc] (:robcee)]

sure, I'll add a patch momentarily.

Comment 2

[Rob Campbell [:rc] (:robcee)]

Attachment: timeouts and whitespace cleanup

This got a little out of control. What started out as a simple change to the shell timeouts morphed into a total cleanup of all the various styles of whitespace in the file. Should me more consistent now.

Comment 3

[Rob Campbell [:rc] (:robcee)]

Attachment: timeouts and cleanup --no-whitespace

emitted with -w for comfort.

Comment 4

[Rob Campbell [:rc] (:robcee)]

cvs commit: Examining .
Checking in master.cfg;
/cvsroot/mozilla/tools/buildbot-configs/testing/sisyphus/master.cfg,v  <--  master.cfg
new revision: 1.12; previous revision: 1.11
done

Comment 5

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 303050 [details] [diff] [review]
patch

failing in public has never looked so good.

Comment 6

[Bob Clary [:bc] (inactive)]

heh.

Checking in public-failures.txt;
/cvsroot/mozilla/js/tests/public-failures.txt,v  <--  public-failures.txt
new revision: 1.32; previous revision: 1.31
done
Checking in test-browser.sh;
/cvsroot/mozilla/js/tests/test-browser.sh,v  <--  test-browser.sh
new revision: 1.7; previous revision: 1.6
done

Comment 7

[(not currently active) Ted Mielczarek]

Mass move of Sisyphus bugs to Testing : Sisyphus. Filter on SisyphusMassMove to ignore.