Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(155)

Unified Diff: infra/bots/gen_tasks.go

Issue 2428603002: Add a buildbotless Android bot (Closed)
Patch Set: Make it a GPU job Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | infra/bots/tasks.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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" {
« no previous file with comments | « no previous file | infra/bots/tasks.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698