Merge lp:~asac/lava-dispatcher/fastmodel-disablesuspend-should-wait-500sec into lp:lava-dispatcher

[Alexander Sack]

500 seconds never failed here, while 240 always failed on fmhackbox in ec2

[Andy Doan (doanac)]

I have a branch that does this using a config variable. Will be merging it soon.

review: Disapprove

[Andy Doan (doanac)]

here's the branch: https://code.launchpad.net/~doanac/lava-dispatcher/fm-fixes

[Alexander Sack (asac)]

please request a merge on that one ... i want this fix by default in..

[Alexander Sack (asac)]

please use 500 for fastmodels. thanks.

[Alexander Sack (asac)]

> please use 500 for fastmodels. thanks.
besides from that the branch looks ok.

Unmerged revisions

367. By Alexander Sack on 2012-08-09

bump disablesuspend timeout to 500 seconds to allow hickups during fast model runs

=== modified file 'lava_dispatcher/client/base.py'
--- lava_dispatcher/client/base.py	2012-08-02 23:05:53 +0000
+++ lava_dispatcher/client/base.py	2012-08-09 20:32:20 +0000
@@ -473,8 +473,8 @@
                 logging.info("Skip raising exception on the home screen has not displayed for health check jobs")
 
         # we need to deal with the time the "input keyevent 82" takes
-        # 240: for fastmodels, 120 failed, 180 failed some
-        session.run('/system/bin/disablesuspend.sh', timeout=240)
+        # 500: for fastmodels never failed for asac on slow, 120 failed, 180 failed some
+        session.run('/system/bin/disablesuspend.sh', timeout=500)
 
     def _enable_network(self):
         session = TesterCommandRunner(self, wait_for_rc=False)