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

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

Issue 2480593002: [Predator] Move time_util from common/ to lib/, split code review related part to code_review_util (Closed)
Patch Set: Created 4 years, 1 month 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 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 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 from google.appengine.ext import ndb 5 from google.appengine.ext import ndb
6 6
7 from common import time_util
8 from common.waterfall import failure_type 7 from common.waterfall import failure_type
8 from lib import time_util
9 from model import analysis_approach_type 9 from model import analysis_approach_type
10 from model.wf_suspected_cl import WfSuspectedCL 10 from model.wf_suspected_cl import WfSuspectedCL
11 from waterfall import build_util 11 from waterfall import build_util
12 12
13 13
14 def GetCLInfo(cl_info_str): 14 def GetCLInfo(cl_info_str):
15 """Gets CL's repo_name and revision.""" 15 """Gets CL's repo_name and revision."""
16 return cl_info_str.split('/') 16 return cl_info_str.split('/')
17 17
18 18
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
132 cl_from_analyzed_build = cl_from_first_failed_build 132 cl_from_analyzed_build = cl_from_first_failed_build
133 133
134 confidence = GetSuspectedCLConfidenceScore( 134 confidence = GetSuspectedCLConfidenceScore(
135 confidences, cl_from_analyzed_build) 135 confidences, cl_from_analyzed_build)
136 approach = ( 136 approach = (
137 analysis_approach_type.TRY_JOB if analysis_approach_type.TRY_JOB in 137 analysis_approach_type.TRY_JOB if analysis_approach_type.TRY_JOB in
138 cl_from_analyzed_build['approaches'] else 138 cl_from_analyzed_build['approaches'] else
139 analysis_approach_type.HEURISTIC) 139 analysis_approach_type.HEURISTIC)
140 140
141 return confidence, approach 141 return confidence, approach
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698