[Findit] Differentiate compile failures and test failures.

appengine/findit/waterfall/detect_first_failure_pipeline.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import base64
 import json
 
+from common import constants
 from common.http_client_appengine import HttpClientAppengine as HttpClient
 from common.pipeline_wrapper import BasePipeline
 from common.pipeline_wrapper import pipeline
+from common.waterfall import failure_type
 from model.wf_analysis import WfAnalysis
 from model.wf_step import WfStep
 from waterfall import build_util
 from waterfall import buildbot
 from waterfall import swarming_util
 
 
 _MAX_BUILDS_TO_CHECK = 20
 
 
@@ skipping 339 matching lines @@
           failed_step, http_client)
 
       if result:  # pragma: no cover
         # Iterates backwards to get a more precise failed_steps info.
         self._UpdateFirstFailureOnTestLevel(
             master_name, builder_name, build_number, step_name,
             failed_step, http_client)
 
     self._UpdateFailureInfoBuilds(failed_steps, builds)
 
+  @staticmethod
+  def _GetFailureType(build_info):
+    if not build_info.failed_steps:
+      return failure_type.UNKNOWN
+    if constants.COMPILE_STEP_NAME in build_info.failed_steps:
+      return failure_type.COMPILE
+    # TODO(http://crbug.com/602733): differentiate test steps from infra ones.
+    return failure_type.TEST
+
   # Arguments number differs from overridden method - pylint: disable=W0221
   def run(self, master_name, builder_name, build_number):
     """
     Args:
       master_name (str): the master name of a build.
       builder_name (str): the builder name of a build.
       build_number (int): the build number of a build.
 
     Returns:
       A dict in the following form:
@@ skipping 31 matching lines @@
             ],
           }
         }
       }
     """
     build_info = self._ExtractBuildInfo(master_name, builder_name, build_number)
 
     if not build_info:  # pragma: no cover
       raise pipeline.Retry('Failed to extract build info.')
 
+    build_failure_type = self._GetFailureType(build_info)
+
     failure_info = {
         'failed': True,
         'master_name': master_name,
         'builder_name': builder_name,
         'build_number': build_number,
         'chromium_revision': build_info.chromium_revision,
         'builds': {},
         'failed_steps': {},
+        'failure_type': build_failure_type,
     }
 
-    if (build_info.result == buildbot.SUCCESS or
-        not build_info.failed_steps):
+    if build_info.result == buildbot.SUCCESS or not build_info.failed_steps:
       failure_info['failed'] = False
       return failure_info
 
     builds = dict()
     self._SaveBlamelistAndChromiumRevisionIntoDict(build_info, builds)
 
     failed_steps = self._CreateADictOfFailedSteps(build_info)
 
     # Checks first failed builds for each failed step.
     self._CheckForFirstKnownFailure(
         master_name, builder_name, build_number, failed_steps, builds)
 
-    # Checks first failed builds for each failed test.
-    self._CheckFirstKnownFailureForSwarmingTests(
-        master_name, builder_name, build_number, failed_steps, builds)
+    if build_failure_type == failure_type.TEST:
+      # Checks first failed builds for each failed test.
+      self._CheckFirstKnownFailureForSwarmingTests(
+          master_name, builder_name, build_number, failed_steps, builds)
 
     failure_info['builds'] = builds
     failure_info['failed_steps'] = failed_steps
 
     analysis = WfAnalysis.Get(master_name, builder_name, build_number)
     analysis.not_passed_steps = build_info.not_passed_steps
+    analysis.build_failure_type = build_failure_type
     analysis.put()
 
     return failure_info