Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(156)

Unified Diff: testing/chromoting/browser_tests_launcher.py

Issue 1718733002: Add a retry to Chromoting tests on Linux builders. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Retain logic to identify failure owing to http://crbug/480025. Add TODO to remove retries for Integ… Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: testing/chromoting/browser_tests_launcher.py
diff --git a/testing/chromoting/browser_tests_launcher.py b/testing/chromoting/browser_tests_launcher.py
index 0b549f33486d6822b3d6205b277736774324ea87..566b3344c37a333d361ab62c26f9f39610d474fe 100644
--- a/testing/chromoting/browser_tests_launcher.py
+++ b/testing/chromoting/browser_tests_launcher.py
@@ -11,6 +11,7 @@ from chromoting_test_utilities import CleanupUserProfileDir
from chromoting_test_utilities import GetJidFromHostLog
from chromoting_test_utilities import GetJidListFromTestResults
from chromoting_test_utilities import InitialiseTestMachineForLinux
+from chromoting_test_utilities import MAX_RETRIES
from chromoting_test_utilities import PrintHostLogContents
from chromoting_test_utilities import PROD_DIR_ID
from chromoting_test_utilities import RunCommandInSubProcess
@@ -23,7 +24,6 @@ FAILING_TESTS = ''
BROWSER_NOT_STARTED_ERROR = (
'Still waiting for the following processes to finish')
TIME_OUT_INDICATOR = '(TIMED OUT)'
-MAX_RETRIES = 1
def LaunchBTCommand(args, command):
@@ -101,10 +101,11 @@ def LaunchBTCommand(args, command):
# A line saying "Still waiting for the following processes to finish",
# and, because sometimes that line gets logged even if the test
# eventually passes, we'll also look for "(TIMED OUT)", before retrying.
- if not (
- BROWSER_NOT_STARTED_ERROR in results and TIME_OUT_INDICATOR in results):
- # Test failed for some other reason. Let's not retry.
- break
+ if BROWSER_NOT_STARTED_ERROR in results and TIME_OUT_INDICATOR in results:
+ print 'Browser-instance not started (http://crbug/480025). Retrying.'
+ else:
+ print 'Test failed for unknown reason. Retrying.'
+
retries += 1
# Check that the test passed.

Powered by Google App Engine
This is Rietveld 408576698