OLD | NEW |
1 # Copyright 2014 The Chromium Authors. All rights reserved. | 1 # Copyright 2014 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 for the Skia PerCommit Housekeeper. | 6 # Recipe for the Skia PerCommit Housekeeper. |
7 | 7 |
8 | 8 |
9 DEPS = [ | 9 DEPS = [ |
| 10 'core', |
10 'recipe_engine/path', | 11 'recipe_engine/path', |
11 'recipe_engine/properties', | 12 'recipe_engine/properties', |
12 'recipe_engine/python', | 13 'recipe_engine/python', |
13 'skia', | |
14 'recipe_engine/step', | 14 'recipe_engine/step', |
| 15 'run', |
| 16 'vars', |
15 ] | 17 ] |
16 | 18 |
17 | 19 |
18 TEST_BUILDERS = { | 20 TEST_BUILDERS = { |
19 'client.skia.fyi': { | 21 'client.skia.fyi': { |
20 'skiabot-linux-housekeeper-000': [ | 22 'skiabot-linux-housekeeper-000': [ |
21 'Housekeeper-PerCommit', | 23 'Housekeeper-PerCommit', |
22 'Housekeeper-PerCommit-Trybot', | 24 'Housekeeper-PerCommit-Trybot', |
23 ], | 25 ], |
24 }, | 26 }, |
25 } | 27 } |
26 | 28 |
27 | 29 |
28 def RunSteps(api): | 30 def RunSteps(api): |
29 # Checkout, compile, etc. | 31 # Checkout, compile, etc. |
30 api.skia.setup() | 32 api.core.setup() |
31 | 33 |
32 cwd = api.path['checkout'] | 34 cwd = api.path['checkout'] |
33 | 35 |
34 api.skia.run( | 36 api.run( |
35 api.step, | 37 api.step, |
36 'android platform self-tests', | 38 'android platform self-tests', |
37 cmd=['python', | 39 cmd=['python', |
38 cwd.join('platform_tools', 'android', 'tests', 'run_all.py')], | 40 cwd.join('platform_tools', 'android', 'tests', 'run_all.py')], |
39 cwd=cwd, | 41 cwd=cwd, |
40 abort_on_failure=False) | 42 abort_on_failure=False) |
41 | 43 |
42 # TODO(borenet): Detect static initializers? | 44 # TODO(borenet): Detect static initializers? |
43 | 45 |
44 gsutil_path = api.path['depot_tools'].join('third_party', 'gsutil', | 46 gsutil_path = api.path['depot_tools'].join('third_party', 'gsutil', |
45 'gsutil') | 47 'gsutil') |
46 if not api.skia.is_trybot: | 48 if not api.vars.is_trybot: |
47 api.skia.run( | 49 api.run( |
48 api.step, | 50 api.step, |
49 'generate and upload doxygen', | 51 'generate and upload doxygen', |
50 cmd=['python', api.skia.resource('generate_and_upload_doxygen.py'), | 52 cmd=['python', api.core.resource('generate_and_upload_doxygen.py'), |
51 gsutil_path], | 53 gsutil_path], |
52 cwd=cwd, | 54 cwd=cwd, |
53 abort_on_failure=False) | 55 abort_on_failure=False) |
54 | 56 |
55 cmd = ['python', api.skia.resource('run_binary_size_analysis.py'), | 57 cmd = ['python', api.core.resource('run_binary_size_analysis.py'), |
56 '--library', api.skia.skia_out.join('Release', 'lib', 'libskia.so'), | 58 '--library', api.vars.skia_out.join( |
| 59 'Release', 'lib', 'libskia.so'), |
57 '--githash', api.properties['revision'], | 60 '--githash', api.properties['revision'], |
58 '--gsutil_path', gsutil_path] | 61 '--gsutil_path', gsutil_path] |
59 if api.skia.is_trybot: | 62 if api.vars.is_trybot: |
60 cmd.extend(['--issue_number', str(api.skia.m.properties['issue'])]) | 63 cmd.extend(['--issue_number', str(api.properties['issue'])]) |
61 api.skia.run( | 64 api.run( |
62 api.step, | 65 api.step, |
63 'generate and upload binary size data', | 66 'generate and upload binary size data', |
64 cmd=cmd, | 67 cmd=cmd, |
65 cwd=cwd, | 68 cwd=cwd, |
66 abort_on_failure=False) | 69 abort_on_failure=False) |
67 | 70 |
68 def GenTests(api): | 71 def GenTests(api): |
69 for mastername, slaves in TEST_BUILDERS.iteritems(): | 72 for mastername, slaves in TEST_BUILDERS.iteritems(): |
70 for slavename, builders_by_slave in slaves.iteritems(): | 73 for slavename, builders_by_slave in slaves.iteritems(): |
71 for buildername in builders_by_slave: | 74 for buildername in builders_by_slave: |
72 test = ( | 75 test = ( |
73 api.test(buildername) + | 76 api.test(buildername) + |
74 api.properties(buildername=buildername, | 77 api.properties(buildername=buildername, |
75 mastername=mastername, | 78 mastername=mastername, |
76 slavename=slavename, | 79 slavename=slavename, |
77 buildnumber=5, | 80 buildnumber=5, |
78 revision='abc123', | 81 revision='abc123', |
79 path_config='kitchen', | 82 path_config='kitchen', |
80 swarm_out_dir='[SWARM_OUT_DIR]') + | 83 swarm_out_dir='[SWARM_OUT_DIR]') + |
81 api.path.exists(api.path['slave_build']) | 84 api.path.exists(api.path['slave_build']) |
82 ) | 85 ) |
83 if 'Trybot' in buildername: | 86 if 'Trybot' in buildername: |
84 test.properties['issue'] = '500' | 87 test.properties['issue'] = '500' |
| 88 test.properties['patchset'] = '1' |
| 89 test.properties['rietveld'] = 'https://codereview.chromium.org' |
85 yield test | 90 yield test |
OLD | NEW |