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

Side by Side Diff: infra/bots/recipes/swarm_perf.py

Issue 2263323002: Apply gerrit ref if it is a Gerrit patch (Closed) Base URL: https://skia.googlesource.com/skia@master
Patch Set: Align Created 4 years, 3 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 5
6 # Recipe module for Skia Swarming perf. 6 # Recipe module for Skia Swarming perf.
7 7
8 8
9 DEPS = [ 9 DEPS = [
10 'build/file', 10 'build/file',
(...skipping 152 matching lines...) Expand 10 before | Expand all | Expand 10 after
163 api.flavor.create_clean_device_dir( 163 api.flavor.create_clean_device_dir(
164 api.flavor.device_dirs.perf_data_dir) 164 api.flavor.device_dirs.perf_data_dir)
165 165
166 # Run nanobench. 166 # Run nanobench.
167 properties = [ 167 properties = [
168 '--properties', 168 '--properties',
169 'gitHash', api.vars.got_revision, 169 'gitHash', api.vars.got_revision,
170 'build_number', api.vars.build_number, 170 'build_number', api.vars.build_number,
171 ] 171 ]
172 if api.vars.is_trybot: 172 if api.vars.is_trybot:
173 properties.extend([ 173 if api.vars.patch_storage == 'gerrit':
174 'issue', api.vars.issue, 174 properties.extend([
175 'patchset', api.vars.patchset, 175 'patch_storage', api.vars.patch_storage,
176 ]) 176 'repository', api.vars.gerrit_repository,
177 'event.patchSet.ref', api.vars.gerrit_ref,
178 'event.change.number', api.vars.gerrit_issue,
179 ])
180 else:
181 properties.extend([
182 'issue', api.vars.issue,
183 'patchset', api.vars.patchset,
184 ])
177 185
178 target = 'nanobench' 186 target = 'nanobench'
179 if 'VisualBench' in api.vars.builder_name: 187 if 'VisualBench' in api.vars.builder_name:
180 target = 'visualbench' 188 target = 'visualbench'
181 args = [ 189 args = [
182 target, 190 target,
183 '--undefok', # This helps branches that may not know new flags. 191 '--undefok', # This helps branches that may not know new flags.
184 '-i', api.flavor.device_dirs.resource_dir, 192 '-i', api.flavor.device_dirs.resource_dir,
185 '--skps', api.flavor.device_dirs.skp_dir, 193 '--skps', api.flavor.device_dirs.skp_dir,
186 '--images', api.flavor.device_path_join( 194 '--images', api.flavor.device_path_join(
(...skipping 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
320 api.path['slave_build'].join('skia', 'infra', 'bots', 'assets', 328 api.path['slave_build'].join('skia', 'infra', 'bots', 'assets',
321 'skimage', 'VERSION'), 329 'skimage', 'VERSION'),
322 api.path['slave_build'].join('skia', 'infra', 'bots', 'assets', 330 api.path['slave_build'].join('skia', 'infra', 'bots', 'assets',
323 'skp', 'VERSION'), 331 'skp', 'VERSION'),
324 api.path['slave_build'].join('skia', 'infra', 'bots', 'assets', 332 api.path['slave_build'].join('skia', 'infra', 'bots', 'assets',
325 'svg', 'VERSION'), 333 'svg', 'VERSION'),
326 api.path['slave_build'].join('tmp', 'uninteresting_hashes.txt') 334 api.path['slave_build'].join('tmp', 'uninteresting_hashes.txt')
327 ) + 335 ) +
328 api.platform('win', 64) 336 api.platform('win', 64)
329 ) 337 )
338
339 gerrit_kwargs = {
340 'patch_storage': 'gerrit',
341 'repository': 'skia',
342 'event.patchSet.ref': 'refs/changes/00/2100/2',
343 'event.change.number': '2100',
344 }
345 yield (
346 api.test('recipe_with_gerrit_patch') +
347 api.properties(
348 buildername='Perf-Ubuntu-GCC-ShuttleA-GPU-GTX550Ti-x86_64-Release-' +
349 'Valgrind-Trybot',
350 mastername='client.skia',
351 slavename='skiabot-linux-swarm-000',
352 buildnumber=5,
353 path_config='kitchen',
354 swarm_out_dir='[SWARM_OUT_DIR]',
355 revision='abc123',
356 **gerrit_kwargs)
357 )
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698