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

Side by Side Diff: appengine/findit/handlers/flake/check_flake.py

Issue 2369333002: [Findit] Capture versionized metadata for master_flake_analysis (Closed)
Patch Set: Created 4 years, 2 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 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.api import users 5 from google.appengine.api import users
6 6
7 from common import constants 7 from common import constants
8 from common.base_handler import BaseHandler 8 from common.base_handler import BaseHandler
9 from common.base_handler import Permission 9 from common.base_handler import Permission
10 from model.analysis_status import STATUS_TO_DESCRIPTION 10 from model.analysis_status import STATUS_TO_DESCRIPTION
(...skipping 26 matching lines...) Expand all
37 'data': { 37 'data': {
38 'error_message': 38 'error_message':
39 ('You could schedule an analysis for flaky test only after ' 39 ('You could schedule an analysis for flaky test only after '
40 'you login with google.com account.'), 40 'you login with google.com account.'),
41 'login_url': self.GetLoginUrl(), 41 'login_url': self.GetLoginUrl(),
42 }, 42 },
43 'return_code': 401, 43 'return_code': 401,
44 } 44 }
45 45
46 data = { 46 data = {
47 'success_rates': [], 47 'pass_rates': [],
48 'analysis_status': STATUS_TO_DESCRIPTION.get( 48 'analysis_status': STATUS_TO_DESCRIPTION.get(
49 master_flake_analysis.status), 49 master_flake_analysis.status),
50 'suspected_flake_build_number': ( 50 'suspected_flake_build_number': (
51 master_flake_analysis.suspected_flake_build_number), 51 master_flake_analysis.suspected_flake_build_number),
52 'master_name': master_name, 52 'master_name': master_name,
53 'builder_name': builder_name, 53 'builder_name': builder_name,
54 'build_number': build_number, 54 'build_number': build_number,
55 'step_name': step_name, 55 'step_name': step_name,
56 'test_name': test_name, 56 'test_name': test_name,
57 } 57 }
58
58 zipped = zip(master_flake_analysis.build_numbers, 59 zipped = zip(master_flake_analysis.build_numbers,
59 master_flake_analysis.success_rates) 60 master_flake_analysis.pass_rates)
stgao 2016/09/28 00:03:24 As this is a rename, be sure to make needed update
60 zipped.sort(key = lambda x: x[0]) 61 zipped.sort(key=lambda x: x[0])
61 for (build_number, success_rate) in zipped: 62
62 data['success_rates'].append([build_number, success_rate]) 63 for (build_number, pass_rate) in zipped:
64 data['pass_rates'].append([build_number, pass_rate])
63 return { 65 return {
64 'template': 'flake/result.html', 66 'template': 'flake/result.html',
65 'data': data 67 'data': data
66 } 68 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698