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/handlers/flake/check_flake.py

Issue 2554143003: [Findit] Using master flake analysis key for updating triage results (Closed)
Patch Set: Addressing comments Created 4 years 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
« no previous file with comments | « no previous file | appengine/findit/handlers/flake/test/check_flake_test.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 import logging 5 import logging
6 6
7 from google.appengine.api import users 7 from google.appengine.api import users
8 from google.appengine.ext import ndb 8 from google.appengine.ext import ndb
9 9
10 from common import auth_util 10 from common import auth_util
(...skipping 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
148 'template': 'error.html', 148 'template': 'error.html',
149 'data': { 149 'data': {
150 'error_message': 'Flake analysis was deleted unexpectedly!', 150 'error_message': 'Flake analysis was deleted unexpectedly!',
151 }, 151 },
152 'return_code': 400 152 'return_code': 400
153 } 153 }
154 154
155 suspected_flake = _GetSuspectedFlakeAnalysisAndTriageResult(analysis) 155 suspected_flake = _GetSuspectedFlakeAnalysisAndTriageResult(analysis)
156 156
157 data = { 157 data = {
158 'key': analysis.key.urlsafe(),
158 'master_name': analysis.master_name, 159 'master_name': analysis.master_name,
159 'builder_name': analysis.builder_name, 160 'builder_name': analysis.builder_name,
160 'build_number': analysis.build_number, 161 'build_number': analysis.build_number,
161 'step_name': analysis.step_name, 162 'step_name': analysis.step_name,
162 'test_name': analysis.test_name, 163 'test_name': analysis.test_name,
163 'pass_rates': [], 164 'pass_rates': [],
164 'analysis_status': analysis.status_description, 165 'analysis_status': analysis.status_description,
165 'version_number': analysis.version_number, 166 'version_number': analysis.version_number,
166 'suspected_flake': suspected_flake, 167 'suspected_flake': suspected_flake,
167 'request_time': time_util.FormatDatetime( 168 'request_time': time_util.FormatDatetime(
(...skipping 28 matching lines...) Expand all
196 data_point.build_number, data_point.pass_rate, None]) 197 data_point.build_number, data_point.pass_rate, None])
197 198
198 # Order by build number from earliest to latest. 199 # Order by build number from earliest to latest.
199 coordinates.sort(key=lambda x: x[0]) 200 coordinates.sort(key=lambda x: x[0])
200 201
201 data['pass_rates'] = coordinates 202 data['pass_rates'] = coordinates
202 return { 203 return {
203 'template': 'flake/result.html', 204 'template': 'flake/result.html',
204 'data': data 205 'data': data
205 } 206 }
OLDNEW
« no previous file with comments | « no previous file | appengine/findit/handlers/flake/test/check_flake_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698