[Findit] Show build analysis references in UI for Findit Cross-platform auto-triage

appengine/findit/handlers/triage_analysis.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.
 
 """This module is to handle manual triage of analysis result.
 
 This handler will flag the analysis result as correct or incorrect.
 TODO: work on an automatic or semi-automatic way to triage analysis result.
 """
 
@@ skipping 28 matching lines @@
 
     or could look like:
 
         (step_name, revision)
   """
   potential_culprit_tuple_list = []
 
   # Iterates through the failures, tests, and suspected_cls, appending potential
   # (step_name, test_name, revision) and (step_name, revision) culprit tuples to
   # the list.
+
   for failure in analysis.result['failures']:
     if failure.get('tests'):
       for test in failure['tests']:
         for suspected_cl in test.get('suspected_cls', []):
           potential_culprit_tuple_list.append((
               failure['step_name'],
               test['test_name'],
               suspected_cl['revision']))
     else:
       for suspected_cl in failure['suspected_cls']:
@@ skipping 22 matching lines @@
 
   # Both analyses must have non-empty potential culprit lists.
   if not potential_culprit_tuple_list_1 or not potential_culprit_tuple_list_2:
     return False
 
   # Both analyses must have matching potential culprit lists.
   return (sorted(potential_culprit_tuple_list_1) ==
           sorted(potential_culprit_tuple_list_2))
 
 
-def _AppendTriageHistoryRecord(analysis, is_correct, user_name):
+def _AppendTriageHistoryRecord(
+    analysis, is_correct, user_name, duplicate=False):
   """Appends a triage history record to the given analysis.
 
   Args:
     analysis: The analysis to which to append the history record.
     correct: True if the history record should indicate a correct judgement,

[chanli, 2016/06/24 18:24:20]

Did you rebase your patch? I remembered you have changed this to 'is_correct'?

         otherwise False.
     user_name: The user_name of the person to include in the triage record.

[chanli, 2016/06/24 18:24:20]

Add description of 'duplicate' here 

[josiahk, 2016/06/27 19:33:25]

Done.

   """
   if is_correct:
     if analysis.suspected_cls:
-      analysis.result_status = result_status.FOUND_CORRECT
+      if duplicate:
+        analysis.result_status = result_status.FOUND_CORRECT_DUPLICATE
+      else:
+        analysis.result_status = result_status.FOUND_CORRECT
       analysis.culprit_cls = analysis.suspected_cls
     else:
       analysis.result_status = result_status.NOT_FOUND_CORRECT
       analysis.culprit_cls = None
   else:
     analysis.culprit_cls = None
     if analysis.suspected_cls:
-      analysis.result_status = result_status.FOUND_INCORRECT
+      if duplicate:
+        analysis.result_status = result_status.FOUND_INCORRECT_DUPLICATE
+      else:
+        analysis.result_status = result_status.FOUND_INCORRECT
     else:
       analysis.result_status = result_status.NOT_FOUND_INCORRECT
 
   triage_record = {
       'triage_timestamp': calendar.timegm(datetime.utcnow().timetuple()),
       'user_name': user_name,
       'result_status': analysis.result_status,
       'version': analysis.version,
   }
   if not analysis.triage_history:
     analysis.triage_history = []
   analysis.triage_history.append(triage_record)
 
   analysis.put()
 
 
 @ndb.transactional
 def _UpdateAnalysisResultStatus(
     master_name, builder_name, build_number, is_correct, user_name=None):
   analysis = WfAnalysis.Get(master_name, builder_name, build_number)
   if not analysis or not analysis.completed:
     return False, None
 
+  analysis.triage_reference_analysis_master_name = None
+  analysis.triage_reference_analysis_builder_name = None
+  analysis.triage_reference_analysis_build_number = None

[chanli, 2016/06/24 18:24:20]

These lines are not needed.

+
   _AppendTriageHistoryRecord(analysis, is_correct, user_name)
 
   return True, analysis
 
 
 def _GetDuplicateAnalyses(original_analysis):
   start_time = (original_analysis.build_start_time -
                 timedelta(hours=MATCHING_ANALYSIS_HOURS_AGO_START))
   end_time = (original_analysis.build_start_time +
               timedelta(hours=MATCHING_ANALYSIS_HOURS_AGO_END))
@@ skipping 15 matching lines @@
   return [analysis for analysis in analysis_results if
           _DoAnalysesMatch(original_analysis, analysis) and
           original_analysis.key is not analysis.key and
           analysis.completed]
 
 
 def _TriageDuplicateResults(original_analysis, is_correct, user_name=None):
   matching_analyses = _GetDuplicateAnalyses(original_analysis)
 
   for analysis in matching_analyses:
-    _AppendTriageHistoryRecord(analysis, is_correct, user_name)
+    analysis.triage_reference_analysis_master_name = (
+        original_analysis.master_name)
+    analysis.triage_reference_analysis_builder_name = (
+        original_analysis.builder_name)
+    analysis.triage_reference_analysis_build_number = (
+        original_analysis.build_number)
+    _AppendTriageHistoryRecord(analysis, is_correct, user_name, duplicate=True)
+
+  return len(matching_analyses)
 
 
 class TriageAnalysis(BaseHandler):
   PERMISSION_LEVEL = Permission.CORP_USER
 
   def HandleGet(self):  # pragma: no cover
     return self.HandlePost()
 
   def HandlePost(self):
     """Sets the manual triage result for the analysis.
 
     Mark the analysis result as correct/wrong/etc.
     TODO: make it possible to set the real culprit CLs.
     """
     url = self.request.get('url').strip()
     build_info = buildbot.ParseBuildUrl(url)
     if not build_info:
       return {'data': {'success': False}}
     master_name, builder_name, build_number = build_info
 
     is_correct = self.request.get('correct').lower() == 'true'
     # As the permission level is CORP_USER, we could assume the current user
     # already logged in.
     user_name = users.get_current_user().email().split('@')[0]
     success, original_analysis = _UpdateAnalysisResultStatus(
         master_name, builder_name, build_number, is_correct, user_name)
+    num_duplicate_analyses = 0
     if success:
-      _TriageDuplicateResults(original_analysis, is_correct, user_name)
-    return {'data': {'success': success}}
+      num_duplicate_analyses = _TriageDuplicateResults(
+          original_analysis, is_correct, user_name)
+    return {'data': {'success': success,
+                     'num_duplicate_analyses': num_duplicate_analyses}}