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

Side by Side Diff: infra/bots/recipes/swarm_test.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 test. 6 # Recipe module for Skia Swarming test.
7 7
8 8
9 DEPS = [ 9 DEPS = [
10 'build/file', 10 'build/file',
(...skipping 412 matching lines...) Expand 10 before | Expand all | Expand 10 after
423 use_hash_file = True 423 use_hash_file = True
424 424
425 # Run DM. 425 # Run DM.
426 properties = [ 426 properties = [
427 'gitHash', api.vars.got_revision, 427 'gitHash', api.vars.got_revision,
428 'master', api.vars.master_name, 428 'master', api.vars.master_name,
429 'builder', api.vars.builder_name, 429 'builder', api.vars.builder_name,
430 'build_number', api.vars.build_number, 430 'build_number', api.vars.build_number,
431 ] 431 ]
432 if api.vars.is_trybot: 432 if api.vars.is_trybot:
433 properties.extend([ 433 if api.vars.patch_storage == 'gerrit':
434 'issue', api.vars.issue, 434 properties.extend([
435 'patchset', api.vars.patchset, 435 'patch_storage', api.vars.patch_storage,
436 ]) 436 'repository', api.vars.gerrit_repository,
437 'event.patchSet.ref', api.vars.gerrit_ref,
438 'event.change.number', api.vars.gerrit_issue,
rmistry 2016/08/23 14:00:41 Actually I do not think this is right. This needs
439 ])
440 else:
441 properties.extend([
442 'issue', api.vars.issue,
443 'patchset', api.vars.patchset,
444 ])
437 445
438 args = [ 446 args = [
439 'dm', 447 'dm',
440 '--undefok', # This helps branches that may not know new flags. 448 '--undefok', # This helps branches that may not know new flags.
441 '--resourcePath', api.flavor.device_dirs.resource_dir, 449 '--resourcePath', api.flavor.device_dirs.resource_dir,
442 '--skps', api.flavor.device_dirs.skp_dir, 450 '--skps', api.flavor.device_dirs.skp_dir,
443 '--images', api.flavor.device_path_join( 451 '--images', api.flavor.device_path_join(
444 api.flavor.device_dirs.images_dir, 'dm'), 452 api.flavor.device_dirs.images_dir, 'dm'),
445 '--colorImages', api.flavor.device_path_join( 453 '--colorImages', api.flavor.device_path_join(
446 api.flavor.device_dirs.images_dir, 'colorspace'), 454 api.flavor.device_dirs.images_dir, 'colorspace'),
(...skipping 406 matching lines...) Expand 10 before | Expand all | Expand 10 after
853 api.path['slave_build'].join('skia', 'infra', 'bots', 'assets', 861 api.path['slave_build'].join('skia', 'infra', 'bots', 'assets',
854 'skimage', 'VERSION'), 862 'skimage', 'VERSION'),
855 api.path['slave_build'].join('skia', 'infra', 'bots', 'assets', 863 api.path['slave_build'].join('skia', 'infra', 'bots', 'assets',
856 'skp', 'VERSION'), 864 'skp', 'VERSION'),
857 api.path['slave_build'].join('skia', 'infra', 'bots', 'assets', 865 api.path['slave_build'].join('skia', 'infra', 'bots', 'assets',
858 'svg', 'VERSION'), 866 'svg', 'VERSION'),
859 api.path['slave_build'].join('tmp', 'uninteresting_hashes.txt') 867 api.path['slave_build'].join('tmp', 'uninteresting_hashes.txt')
860 ) + 868 ) +
861 api.platform('win', 64) 869 api.platform('win', 64)
862 ) 870 )
871
872 builder = 'Test-Ubuntu-GCC-GCE-CPU-AVX2-x86-Debug-Trybot'
873 gerrit_kwargs = {
874 'patch_storage': 'gerrit',
875 'repository': 'skia',
876 'event.patchSet.ref': 'refs/changes/00/2100/2',
877 'event.change.number': '2100',
878 }
879 yield (
880 api.test('recipe_with_gerrit_patch') +
881 api.properties(
882 buildername=builder,
883 mastername='client.skia',
884 slavename='skiabot-linux-swarm-000',
885 buildnumber=5,
886 path_config='kitchen',
887 swarm_out_dir='[SWARM_OUT_DIR]',
888 revision='abc123',
889 **gerrit_kwargs)
890 )
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698