aboutsummaryrefslogtreecommitdiffhomepage
path: root/infra/bots/recipe_modules/vars/api.py
diff options
context:
space:
mode:
authorGravatar Eric Boren <borenet@google.com>2018-04-17 11:28:46 +0000
committerGravatar Skia Commit-Bot <skia-commit-bot@chromium.org>2018-04-17 11:28:54 +0000
commitbd2e1f1ca6f511e567020bebd7479cade8e246a1 (patch)
tree754a446c7fcd806b79856de87d7682f983b0c128 /infra/bots/recipe_modules/vars/api.py
parentf39bbe5a31542396a8aabf8066ee46862312d7d8 (diff)
Revert "Reland "[infra] Run recipes through Kitchen""
This reverts commit f39bbe5a31542396a8aabf8066ee46862312d7d8. Reason for revert: Broke lots of bots Original change's description: > Reland "[infra] Run recipes through Kitchen" > > Bug: skia:7050 > Change-Id: I331014c408dd5b6f7baafd78ffd12f1235c5bfe7 > Reviewed-on: https://skia-review.googlesource.com/121481 > Commit-Queue: Eric Boren <borenet@google.com> > Reviewed-by: Ben Wagner <benjaminwagner@google.com> > Reviewed-by: Ravi Mistry <rmistry@google.com> TBR=borenet@google.com,benjaminwagner@google.com,rmistry@google.com Change-Id: I4548c66a9cb0b4f105e2bc3b1c7185d14549172f No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: skia:7050 Reviewed-on: https://skia-review.googlesource.com/121463 Reviewed-by: Eric Boren <borenet@google.com> Commit-Queue: Eric Boren <borenet@google.com>
Diffstat (limited to 'infra/bots/recipe_modules/vars/api.py')
-rw-r--r--infra/bots/recipe_modules/vars/api.py15
1 files changed, 4 insertions, 11 deletions
diff --git a/infra/bots/recipe_modules/vars/api.py b/infra/bots/recipe_modules/vars/api.py
index 26bde8c9d8..9158233213 100644
--- a/infra/bots/recipe_modules/vars/api.py
+++ b/infra/bots/recipe_modules/vars/api.py
@@ -27,12 +27,6 @@ class SkiaVarsApi(recipe_api.RecipeApi):
self.builder_name = self.m.properties['buildername']
self.slave_dir = self.m.path['start_dir']
-
- # Special input/output directories.
- self.build_dir = self.slave_dir.join('build')
- self.test_dir = self.slave_dir.join('test')
- self.perf_dir = self.slave_dir.join('perf')
-
self.checkout_root = self.slave_dir
self.default_env = self.m.context.env
self.default_env['CHROME_HEADLESS'] = '1'
@@ -40,7 +34,7 @@ class SkiaVarsApi(recipe_api.RecipeApi):
self.default_env.get('PATH', '%(PATH)s'),
str(self.m.bot_update._module.PACKAGE_REPO_ROOT),
])
- self.gclient_env = {'DEPOT_TOOLS_UPDATE': '0'}
+ self.gclient_env = {}
self.is_compile_bot = self.builder_name.startswith('Build-')
self.persistent_checkout = False
@@ -70,7 +64,7 @@ class SkiaVarsApi(recipe_api.RecipeApi):
# got_revision is filled in after checkout steps.
self.got_revision = None
else:
- # If there's no persistent checkout, then we have to assume we got the
+ # If there's no persistent checkout, then we have to asume we got the
# correct revision of the files from isolate.
self.got_revision = self.m.properties['revision']
@@ -96,8 +90,7 @@ class SkiaVarsApi(recipe_api.RecipeApi):
self.resource_dir = self.skia_dir.join('resources')
self.images_dir = self.slave_dir.join('skimage')
self.skia_out = self.skia_dir.join('out', self.builder_name)
- self.swarming_out_dir = self.slave_dir.join(
- self.m.properties['swarm_out_dir'])
+ self.swarming_out_dir = self.make_path(self.m.properties['swarm_out_dir'])
if 'ParentRevision' in self.builder_name:
# Tasks that depend on ParentRevision builds usually also depend on a
# second build task. Use a different path for build results so that the
@@ -106,7 +99,7 @@ class SkiaVarsApi(recipe_api.RecipeApi):
self.local_skp_dir = self.slave_dir.join('skp')
self.local_svg_dir = self.slave_dir.join('svg')
if not self.is_compile_bot:
- self.skia_out = self.build_dir.join('out')
+ self.skia_out = self.slave_dir.join('out')
self.tmp_dir = self.m.path['start_dir'].join('tmp')
# Some bots also require a checkout of chromium.