1436a09e1f
Break Skia recipe module into:
- skia_vars: defines and stores variables (eg. paths)
- skia_step: utilities for running Skia steps
- skia_flavor: flavor-specific stuff
- skia: top-level setup, checkout, test/perf steps, etc etc
This establishes a saner dependency structure for the recipes; skia_vars
is at the bottom level, skia_step depends on it, skia_flavor depends on
both of them, skia depends on all of the above, and the recipes
themselves may depend on any or all of them.
Next steps:
- Merge buildbot_spec into skia_vars
- Move test_steps and perf_steps from skia recipe_module into
swarm_test and swarm_perf recipes
- Cleaner checkout_steps process
BUG=skia:5578
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2198173002
Review-Url: https://codereview.chromium.org/2198173002
91 lines
2.5 KiB
Python
91 lines
2.5 KiB
Python
# Copyright 2014 The Chromium Authors. All rights reserved.
|
|
# Use of this source code is governed by a BSD-style license that can be
|
|
# found in the LICENSE file.
|
|
|
|
|
|
# Recipe for the Skia PerCommit Housekeeper.
|
|
|
|
|
|
DEPS = [
|
|
'core',
|
|
'recipe_engine/path',
|
|
'recipe_engine/properties',
|
|
'recipe_engine/python',
|
|
'recipe_engine/step',
|
|
'run',
|
|
'vars',
|
|
]
|
|
|
|
|
|
TEST_BUILDERS = {
|
|
'client.skia.fyi': {
|
|
'skiabot-linux-housekeeper-000': [
|
|
'Housekeeper-PerCommit',
|
|
'Housekeeper-PerCommit-Trybot',
|
|
],
|
|
},
|
|
}
|
|
|
|
|
|
def RunSteps(api):
|
|
# Checkout, compile, etc.
|
|
api.core.setup()
|
|
|
|
cwd = api.path['checkout']
|
|
|
|
api.run(
|
|
api.step,
|
|
'android platform self-tests',
|
|
cmd=['python',
|
|
cwd.join('platform_tools', 'android', 'tests', 'run_all.py')],
|
|
cwd=cwd,
|
|
abort_on_failure=False)
|
|
|
|
# TODO(borenet): Detect static initializers?
|
|
|
|
gsutil_path = api.path['depot_tools'].join('third_party', 'gsutil',
|
|
'gsutil')
|
|
if not api.vars.is_trybot:
|
|
api.run(
|
|
api.step,
|
|
'generate and upload doxygen',
|
|
cmd=['python', api.core.resource('generate_and_upload_doxygen.py'),
|
|
gsutil_path],
|
|
cwd=cwd,
|
|
abort_on_failure=False)
|
|
|
|
cmd = ['python', api.core.resource('run_binary_size_analysis.py'),
|
|
'--library', api.vars.skia_out.join(
|
|
'Release', 'lib', 'libskia.so'),
|
|
'--githash', api.properties['revision'],
|
|
'--gsutil_path', gsutil_path]
|
|
if api.vars.is_trybot:
|
|
cmd.extend(['--issue_number', str(api.properties['issue'])])
|
|
api.run(
|
|
api.step,
|
|
'generate and upload binary size data',
|
|
cmd=cmd,
|
|
cwd=cwd,
|
|
abort_on_failure=False)
|
|
|
|
def GenTests(api):
|
|
for mastername, slaves in TEST_BUILDERS.iteritems():
|
|
for slavename, builders_by_slave in slaves.iteritems():
|
|
for buildername in builders_by_slave:
|
|
test = (
|
|
api.test(buildername) +
|
|
api.properties(buildername=buildername,
|
|
mastername=mastername,
|
|
slavename=slavename,
|
|
buildnumber=5,
|
|
revision='abc123',
|
|
path_config='kitchen',
|
|
swarm_out_dir='[SWARM_OUT_DIR]') +
|
|
api.path.exists(api.path['slave_build'])
|
|
)
|
|
if 'Trybot' in buildername:
|
|
test.properties['issue'] = '500'
|
|
test.properties['patchset'] = '1'
|
|
test.properties['rietveld'] = 'https://codereview.chromium.org'
|
|
yield test
|