a6f07c5b5c
This is a reland of 50b1248ed8
Fixes from the original are in PS2. Change several recipes to use the
shared 'is_trybot' variable, and explicitly check for 0 rather than
relying on patch_ref not being set.
Original change's description:
> Use integer patchset and issue variables.
>
> See https://skia-review.googlesource.com/c/buildbot/+/246397/
>
> Change-Id: I39dae809abdc63007f65bc05040809d1a870b118
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/246302
> Commit-Queue: Ben Wagner aka dogben <benjaminwagner@google.com>
> Reviewed-by: Eric Boren <borenet@google.com>
Change-Id: Ibe1d4f0dc1dc61c39eee6c980bd7509fc260101b
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/246919
Commit-Queue: Eric Boren <borenet@google.com>
Auto-Submit: Ben Wagner aka dogben <benjaminwagner@google.com>
Reviewed-by: Eric Boren <borenet@google.com>
77 lines
2.1 KiB
Python
77 lines
2.1 KiB
Python
# Copyright 2016 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 uploading buildstats results to Perf.
|
|
|
|
|
|
DEPS = [
|
|
'recipe_engine/context',
|
|
'recipe_engine/file',
|
|
'recipe_engine/path',
|
|
'recipe_engine/properties',
|
|
'recipe_engine/step',
|
|
'recipe_engine/time',
|
|
'vars',
|
|
]
|
|
|
|
|
|
def RunSteps(api):
|
|
# Upload the buildstats results.
|
|
api.vars.setup()
|
|
|
|
now = api.time.utcnow()
|
|
src_path = api.path['start_dir'].join('perf')
|
|
with api.context(cwd=src_path):
|
|
results = api.file.glob_paths(
|
|
'find results',
|
|
src_path,
|
|
'*.json',
|
|
test_data=['buildstats_abc123.json', 'buildstats_def.json'])
|
|
if not len(results): # pragma: nocover
|
|
raise Exception('Unable to find buildstats JSON file!')
|
|
|
|
for src in results:
|
|
basename = api.path.basename(src)
|
|
basename = api.properties['revision'] + '_' + basename
|
|
gs_path = '/'.join((
|
|
'buildstats-json-v1', str(now.year).zfill(4),
|
|
str(now.month).zfill(2), str(now.day).zfill(2), str(now.hour).zfill(2),
|
|
api.vars.builder_name))
|
|
|
|
if api.vars.is_trybot:
|
|
gs_path = '/'.join(('trybot', gs_path,
|
|
str(api.vars.issue), str(api.vars.patchset)))
|
|
|
|
dst = '/'.join((
|
|
'gs://%s' % api.properties['gs_bucket'], gs_path, basename))
|
|
|
|
api.step(
|
|
'upload %s' % src,
|
|
cmd=['gsutil', 'cp', '-z', 'json', src, dst],
|
|
infra_step=True)
|
|
|
|
|
|
def GenTests(api):
|
|
builder = 'BuildStats-Debian9-EMCC-wasm-Release-PathKit'
|
|
yield (
|
|
api.test('normal_bot') +
|
|
api.properties(buildername=builder,
|
|
gs_bucket='skia-perf',
|
|
revision='abc123',
|
|
path_config='kitchen')
|
|
)
|
|
|
|
yield (
|
|
api.test('trybot') +
|
|
api.properties.tryserver(
|
|
gerrit_project='skia',
|
|
gerrit_url='https://skia-review.googlesource.com/',
|
|
) +
|
|
api.properties(buildername=builder,
|
|
gs_bucket='skia-perf',
|
|
revision='abc123',
|
|
path_config='kitchen')
|
|
)
|