diff options
author | 2017-04-06 12:53:31 +0000 | |
---|---|---|
committer | 2017-04-06 12:53:42 +0000 | |
commit | 6276a7c9996e9b8d834992dbaa21880f50fe7601 (patch) | |
tree | 6e805e16b9935fa0adda6b2dab7c868e1e2b947e /infra/bots/gen_tasks.go | |
parent | 93b1b4feda10c50ce10763ddf7e807f9b85c7953 (diff) |
Revert "Use bundled recipes on all Test/Perf tasks"
This reverts commit 43b9c6bbf66b0927a99062c68dff9ea8358f82db.
Reason for revert: Windows failing
Original change's description:
> Use bundled recipes on all Test/Perf tasks
>
> It won't save as much time for faster machines, but it should still
> shave a few seconds, and it'll be nice to be consistent.
>
> Bug: skia:5813
> Change-Id: I5724e7aae83851edff3129265a9ffee5c3f95825
> Reviewed-on: https://skia-review.googlesource.com/11340
> Commit-Queue: Eric Boren <borenet@google.com>
> Reviewed-by: Kevin Lubick <kjlubick@google.com>
>
TBR=borenet@google.com,kjlubick@google.com,reviews@skia.org,iannucci@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: I2a9cc1c12079755969879e37da5cae9f93921ddd
Reviewed-on: https://skia-review.googlesource.com/11416
Reviewed-by: Eric Boren <borenet@google.com>
Commit-Queue: Eric Boren <borenet@google.com>
Diffstat (limited to 'infra/bots/gen_tasks.go')
-rw-r--r-- | infra/bots/gen_tasks.go | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/infra/bots/gen_tasks.go b/infra/bots/gen_tasks.go index 71a53a2a97..0301034cbd 100644 --- a/infra/bots/gen_tasks.go +++ b/infra/bots/gen_tasks.go @@ -244,13 +244,6 @@ func bundleRecipes(b *specs.TasksCfgBuilder) string { return BUNDLE_RECIPES_NAME } -// useBundledRecipes returns true iff the given bot should use bundled recipes -// instead of syncing recipe DEPS itself. -func useBundledRecipes(parts map[string]string) bool { - // Use bundled recipes for all test/perf tasks. - return true -} - // compile generates a compile task. Returns the name of the last task in the // generated chain of tasks, which the Job should add as a dependency. func compile(b *specs.TasksCfgBuilder, name string, parts map[string]string) string { @@ -484,7 +477,7 @@ func test(b *specs.TasksCfgBuilder, name string, parts map[string]string, compil MaxAttempts: 1, Priority: 0.8, } - if useBundledRecipes(parts) { + if parts["os"] == "Android" { s.Dependencies = append(s.Dependencies, BUNDLE_RECIPES_NAME) s.Isolate = "test_skia_bundled.isolate" } @@ -534,10 +527,10 @@ func perf(b *specs.TasksCfgBuilder, name string, parts map[string]string, compil if strings.Contains(parts["extra_config"], "Skpbench") { recipe = "swarm_skpbench" isolate = "skpbench_skia.isolate" - if useBundledRecipes(parts) { + if parts["os"] == "Android" { isolate = "skpbench_skia_bundled.isolate" } - } else if useBundledRecipes(parts) { + } else if parts["os"] == "Android" { isolate = "perf_skia_bundled.isolate" } s := &specs.TaskSpec{ @@ -565,7 +558,7 @@ func perf(b *specs.TasksCfgBuilder, name string, parts map[string]string, compil MaxAttempts: 1, Priority: 0.8, } - if useBundledRecipes(parts) { + if parts["os"] == "Android" { s.Dependencies = append(s.Dependencies, BUNDLE_RECIPES_NAME) } if strings.Contains(parts["extra_config"], "Valgrind") { |