Chromium Code Reviews| Index: infra/bots/gen_tasks.go |
| diff --git a/infra/bots/gen_tasks.go b/infra/bots/gen_tasks.go |
| index b81950dcd28f923ad5adb5a740faca50617f13df..b52e065f26e721a2d458484b7807829030a03fee 100644 |
| --- a/infra/bots/gen_tasks.go |
| +++ b/infra/bots/gen_tasks.go |
| @@ -43,6 +43,7 @@ var ( |
| JOBS = []string{ |
| "Build-Ubuntu-GCC-x86_64-Release-GN", |
| "Perf-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-GN", |
| + "Test-Android-Clang-AndroidOne-GPU-Mali400MP2-arm-Release-GN_Android", |
| "Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-GN", |
| "Housekeeper-PerCommit-InfraTests", |
| } |
| @@ -83,7 +84,7 @@ func deriveCompileTaskName(jobName string, parts map[string]string) string { |
| } else if !strings.Contains(ec, "GN_Android") { |
| ec = task_os |
| } |
| - task_os = "Android" |
| + task_os = "Ubuntu" |
| } else if task_os == "iOS" { |
| ec = task_os |
| task_os = "Mac" |
| @@ -475,6 +476,15 @@ func process(cfg *specs.TasksCfg, name string) { |
| // Any remaining bots need a compile task. |
| compileTaskName := deriveCompileTaskName(name, parts) |
| + compileTaskParts, err := jobNameSchema.ParseJobName(compileTaskName) |
| + if err != nil { |
| + glog.Fatal(err) |
| + } |
| + // Temporarily disable the Housekeeper's compile Task, since we aren't |
| + // yet running that Job. |
| + if parts["role"] != "Housekeeper" { |
|
dogben
2016/10/17 16:32:31
Should this just be name != "Housekeeper-PerCommit
borenet
2016/10/17 16:51:03
So we need this if-statement just to prevent us fr
|
| + compile(cfg, compileTaskName, compileTaskParts) |
| + } |
| // Housekeeper. |
| if parts["role"] == "Housekeeper" && name != "Housekeeper-PerCommit-InfraTests" { |