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

Side by Side Diff: infra/bots/recipe_modules/vars/api.py

Issue 2382313003: Stop uploading nanobench results on Valgrind bot (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
« no previous file with comments | « no previous file | infra/bots/recipes/swarm_perf.expected/Perf-Ubuntu-GCC-ShuttleA-GPU-GTX550Ti-x86_64-Release-Valgrind.json » ('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 5
6 # pylint: disable=W0201 6 # pylint: disable=W0201
7 7
8 8
9 from recipe_engine import recipe_api 9 from recipe_engine import recipe_api
10 10
(...skipping 126 matching lines...) Expand 10 before | Expand all | Expand 10 after
137 upload_dm_results = True 137 upload_dm_results = True
138 for s in skip_upload_bots: 138 for s in skip_upload_bots:
139 if s in self.m.properties['buildername']: 139 if s in self.m.properties['buildername']:
140 upload_dm_results = False 140 upload_dm_results = False
141 break 141 break
142 return upload_dm_results 142 return upload_dm_results
143 143
144 @property 144 @property
145 def upload_perf_results(self): 145 def upload_perf_results(self):
146 # TODO(borenet): Move this into the swarm_perf recipe. 146 # TODO(borenet): Move this into the swarm_perf recipe.
147 return ('Release' in self.m.properties['buildername']) 147 if 'Release' not in self.m.properties['buildername']:
148 return False
149 skip_upload_bots = [
150 'ASAN',
151 'Coverage',
152 'MSAN',
153 'TSAN',
154 'UBSAN',
155 'Valgrind',
156 ]
157 upload_perf_results = True
158 for s in skip_upload_bots:
159 if s in self.m.properties['buildername']:
160 upload_perf_results = False
161 break
162 return upload_perf_results
OLDNEW
« no previous file with comments | « no previous file | infra/bots/recipes/swarm_perf.expected/Perf-Ubuntu-GCC-ShuttleA-GPU-GTX550Ti-x86_64-Release-Valgrind.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698