commit 9eb333def4417ed220984a2e1224ef4db8590dbb
author Chris Sosa <sosa@chromium.org> Wed Jul 17 10:00:25 2013 -0700
committer ChromeBot <chrome-bot@google.com> Wed Jul 17 13:11:33 2013 -0700
tree 678dc23d9ef54f9bdb4dccf1cbc6d015a254c709
parent b8c2af54d3fbd9f5497e4a205e90ba89da939869

Add more protection against flakiness of VM tests on builders.

BUG=chromium:260036
TEST=Ran it, pyflakes, pylint.

Change-Id: Id1b08673eb28f700e43c50a5260358f093ddf333
Reviewed-on: https://gerrit.chromium.org/gerrit/62351
Commit-Queue: Chris Sosa <sosa@chromium.org>
Reviewed-by: Chris Sosa <sosa@chromium.org>
Tested-by: Chris Sosa <sosa@chromium.org>

devmode-test/devinstall_test.py

diff --git a/devmode-test/devinstall_test.py b/devmode-test/devinstall_test.py
index ca19e5f..8094e0c 100755
--- a/devmode-test/devinstall_test.py
+++ b/devmode-test/devinstall_test.py
@@ -20,6 +20,7 @@
 import socket
 import sys
 import tempfile
+import time
 
 import constants
 sys.path.append(constants.SOURCE_ROOT)
@@ -35,7 +36,8 @@
 _LOCALHOST = 'localhost'
 _PRIVATE_KEY = os.path.join(constants.CROSUTILS_DIR, 'mod_for_test_scripts',
                             'ssh_keys', 'testing_rsa')
-_MAX_SSH_ATTEMPTS = 3
+_MAX_SSH_ATTEMPTS = 5
+_TIME_BETWEEN_ATTEMPT = 30
 
 class TestError(Exception):
   """Raised on any error during testing. It being raised is a test failure."""
@@ -135,8 +137,9 @@
         return
       except cros_build_lib.RunCommandError as e:
         logging.warning('Failed to connect to VM')
-        logging.debug(e)
+        logging.warning(e)
         self._StopVM()
+        time.sleep(_TIME_BETWEEN_ATTEMPT)
     else:
       raise TestError('Max attempts to connect to VM exceeded')