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

Side by Side Diff: appengine/findit/waterfall/build_failure_analysis.py

Issue 2344443005: [Findit] Factoring the gitiles (etc) stuff out into its own directory (Closed)
Patch Set: rebase-update Created 4 years, 3 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 unified diff | Download patch
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import collections 5 import collections
6 import os 6 import os
7 import re 7 import re
8 8
9 from common.diff import ChangeType 9 from gitiles.diff import ChangeType
10 from common.git_repository import GitRepository 10 from gitiles.git_repository import GitRepository
11 from common.http_client_appengine import HttpClientAppengine as HttpClient 11 from common.http_client_appengine import HttpClientAppengine as HttpClient
12 from waterfall import waterfall_config 12 from waterfall import waterfall_config
13 from waterfall.failure_signal import FailureSignal 13 from waterfall.failure_signal import FailureSignal
14 14
15 15
16 def _IsSameFile(changed_src_file_path, file_path_in_log): 16 def _IsSameFile(changed_src_file_path, file_path_in_log):
17 """Guesses if the two files are the same. 17 """Guesses if the two files are the same.
18 18
19 Args: 19 Args:
20 changed_src_file_path (str): Full path of a file committed to git repo. 20 changed_src_file_path (str): Full path of a file committed to git repo.
(...skipping 706 matching lines...) Expand 10 before | Expand all | Expand 10 after
727 continue 727 continue
728 728
729 step_analysis_result['suspected_cls'].append( 729 step_analysis_result['suspected_cls'].append(
730 CreateCLInfoDict(justification_dict, build_number, 730 CreateCLInfoDict(justification_dict, build_number,
731 change_logs[revision])) 731 change_logs[revision]))
732 732
733 # TODO(stgao): sort CLs by score. 733 # TODO(stgao): sort CLs by score.
734 analysis_result['failures'].append(step_analysis_result) 734 analysis_result['failures'].append(step_analysis_result)
735 735
736 return analysis_result 736 return analysis_result
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698