Add new isolate specifically for lottie-ci
This should fix the Test.+LottieWeb jobs (which live in
the lottie-ci repo) by including the lottie folder which
is brought in via DEPS and gclient sync.
Change-Id: Iffcd333e4113d95cfa2c9d92bc87aec527cb3f8c
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/300637
Auto-Submit: Kevin Lubick <kjlubick@google.com>
Reviewed-by: Eric Boren <borenet@google.com>
diff --git a/infra/bots/gen_tasks_logic/gen_tasks_logic.go b/infra/bots/gen_tasks_logic/gen_tasks_logic.go
index cebed61..c250236 100644
--- a/infra/bots/gen_tasks_logic/gen_tasks_logic.go
+++ b/infra/bots/gen_tasks_logic/gen_tasks_logic.go
@@ -1303,7 +1303,11 @@
isolate = "canvaskit.isolate"
recipe = "test_canvaskit"
} else if b.extraConfig("LottieWeb") {
- isolate = "lottie_web.isolate"
+ // lottie_ci.isolate differs from lottie_web.isolate in that it includes more of the files,
+ // especially those brought in via DEPS in the lottie-ci repo. The main difference between
+ // Perf.+LottieWeb and Test.+LottieWeb is that the former pulls in the lottie build via
+ // npm and the latter always tests at lottie's ToT.
+ isolate = "lottie_ci.isolate"
recipe = "test_lottie_web"
}
b.recipeProp("gold_hashes_url", b.cfg.GoldHashesURL)
diff --git a/infra/bots/lottie_ci.isolate b/infra/bots/lottie_ci.isolate
new file mode 100644
index 0000000..09b7249
--- /dev/null
+++ b/infra/bots/lottie_ci.isolate
@@ -0,0 +1,13 @@
+{
+ 'includes': [
+ 'swarm_recipe.isolate',
+ ],
+ 'variables': {
+ 'files': [
+ '../../../lottie',
+ '../../tools/lottie-web-perf',
+ '../../tools/lottiecap',
+ '../lottiecap',
+ ],
+ },
+}
diff --git a/infra/bots/lottie_web.isolate b/infra/bots/lottie_web.isolate
index 77f37f1..eeb475e 100644
--- a/infra/bots/lottie_web.isolate
+++ b/infra/bots/lottie_web.isolate
@@ -5,8 +5,6 @@
'variables': {
'files': [
'../../tools/lottie-web-perf',
- '../../tools/lottiecap',
- '../lottiecap',
],
},
}