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

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

Issue 2376573004: [Findit] For automatic analyses of flaky tests, run the Swarming tasks off PST peak hours. (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 10 matching lines...) Expand all
21 builder_name = self.request.get('builder_name').strip() 21 builder_name = self.request.get('builder_name').strip()
22 build_number = int(self.request.get('build_number').strip()) 22 build_number = int(self.request.get('build_number').strip())
23 step_name = self.request.get('step_name').strip() 23 step_name = self.request.get('step_name').strip()
24 test_name = self.request.get('test_name').strip() 24 test_name = self.request.get('test_name').strip()
25 force = (users.is_current_user_admin() and 25 force = (users.is_current_user_admin() and
26 self.request.get('force') == '1') 26 self.request.get('force') == '1')
27 allow_new_analysis = self.IsCorpUserOrAdmin() 27 allow_new_analysis = self.IsCorpUserOrAdmin()
28 28
29 master_flake_analysis = ScheduleAnalysisIfNeeded( 29 master_flake_analysis = ScheduleAnalysisIfNeeded(
30 master_name, builder_name, build_number, step_name, test_name, 30 master_name, builder_name, build_number, step_name, test_name,
31 allow_new_analysis, force=force, 31 allow_new_analysis, force=force, manually_triggered=True,
32 queue_name=constants.WATERFALL_ANALYSIS_QUEUE) 32 queue_name=constants.WATERFALL_ANALYSIS_QUEUE)
33 33
34 if not master_flake_analysis: # pragma: no cover. 34 if not master_flake_analysis: # pragma: no cover.
35 return { 35 return {
36 'template': 'error.html', 36 'template': 'error.html',
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(),
(...skipping 15 matching lines...) Expand all
57 } 57 }
58 zipped = zip(master_flake_analysis.build_numbers, 58 zipped = zip(master_flake_analysis.build_numbers,
59 master_flake_analysis.success_rates) 59 master_flake_analysis.success_rates)
60 zipped.sort(key = lambda x: x[0]) 60 zipped.sort(key = lambda x: x[0])
61 for (build_number, success_rate) in zipped: 61 for (build_number, success_rate) in zipped:
62 data['success_rates'].append([build_number, success_rate]) 62 data['success_rates'].append([build_number, success_rate])
63 return { 63 return {
64 'template': 'flake/result.html', 64 'template': 'flake/result.html',
65 'data': data 65 'data': data
66 } 66 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698