Archive retry summary for layout tests along with layout test results.

scripts/slave/recipe_modules/test_utils/api.py

Index: scripts/slave/recipe_modules/test_utils/api.py
diff --git a/scripts/slave/recipe_modules/test_utils/api.py b/scripts/slave/recipe_modules/test_utils/api.py
index eefa6192145af84abde843bc1fab202f42ea3579..aad3f3c944c8c416b33f38ba3cdf2a3acc0b8eb4 100644
--- a/scripts/slave/recipe_modules/test_utils/api.py
+++ b/scripts/slave/recipe_modules/test_utils/api.py
@@ -195,6 +195,26 @@ class TestUtilsApi(recipe_api.RecipeApi):
       elif ignored_failures:
         self.m.step.active_result.presentation.status = self.m.step.WARNING
 
+    if test.name == 'webkit_tests':
+        self._archive_retry_summary({
+            'failures': sorted(new_failures),
+            'ignored': sorted(ignored_failures)
+        })
+
+  def _archive_retry_summary(self, retry_summary):
+    """Archives the retry summary as JSON, storing it alongside the results
+    from the first run."""
+    script = self.m.chromium.package_repo_resource(
+        'scripts', 'slave', 'chromium', 'archive_layout_test_retry_summary.py')
+    args = [
+        '--retry-summary', self.m.json.dumps(retry_summary),

[Dirk Pranke, 2016/10/15 22:34:36]

Passing this as a command line argument seems awkward; I'd write it to a file
instead.

+        '--build-number', self.m.properties['buildnumber'],
+        '--builder-name', self.m.properties['buildername'],
+        '--gs-bucket', 'gs://chromium-layout-test-archives',
+        '--staging-dir', self.m.path['cache'].join('chrome_staging'),
+    ]
+    self.m.python('archive_retry_summary', script, args)
+
   def create_results_from_json(self, data):
     return TestResults(data)
 
@@ -219,4 +239,3 @@ class TestUtilsApi(recipe_api.RecipeApi):
     The test_results will be an instance of the GTestResults class.
     """
     return GTestResultsOutputPlaceholder(self, add_json_log)
-