infra: In milo, split console into 'compile' and 'test'.

This will provide an easier-to-read summary for wranglers because
test failures will be grouped instead of spread out in between
compile builders.

Bug: angleproject:6496
Change-Id: I45e2f1a868d887de22133684aa0b4821ed4c8f84
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/3233900
Commit-Queue: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>

infra/config/generated/luci-milo.cfg

@@ -12,92 +12,92 @@ consoles {
   manifest_name: "REVISION"
   builders {
     name: "buildbucket/luci.angle.ci/android-arm-compile"
-    category: "angle|android|clang|arm"
+    category: "compile|android|clang|arm"
     short_name: "rel"
   }
   builders {
     name: "buildbucket/luci.angle.ci/android-arm-dbg-compile"
-    category: "angle|android|clang|arm"
+    category: "compile|android|clang|arm"
     short_name: "dbg"
   }
   builders {
     name: "buildbucket/luci.angle.ci/android-arm64-dbg-compile"
-    category: "angle|android|clang|arm64"
+    category: "compile|android|clang|arm64"
     short_name: "dbg"
   }
   builders {
     name: "buildbucket/luci.angle.ci/android-arm64-test"
-    category: "angle|android|clang|arm64"
+    category: "test|android|clang|arm64"
     short_name: "rel"
   }
   builders {
     name: "buildbucket/luci.angle.ci/linux-dbg-compile"
-    category: "angle|linux|clang|x64"
+    category: "compile|linux|clang|x64"
     short_name: "dbg"
   }
   builders {
     name: "buildbucket/luci.angle.ci/linux-test"
-    category: "angle|linux|clang|x64"
+    category: "test|linux|clang|x64"
     short_name: "rel"
   }
   builders {
     name: "buildbucket/luci.angle.ci/mac-dbg-compile"
-    category: "angle|mac|clang|x64"
+    category: "compile|mac|clang|x64"
     short_name: "dbg"
   }
   builders {
     name: "buildbucket/luci.angle.ci/mac-test"
-    category: "angle|mac|clang|x64"
+    category: "test|mac|clang|x64"
     short_name: "rel"
   }
   builders {
     name: "buildbucket/luci.angle.ci/win-dbg-compile"
-    category: "angle|win|clang|x64"
+    category: "compile|win|clang|x64"
     short_name: "dbg"
   }
   builders {
     name: "buildbucket/luci.angle.ci/win-msvc-compile"
-    category: "angle|win|msvc|x64"
+    category: "compile|win|msvc|x64"
     short_name: "rel"
   }
   builders {
     name: "buildbucket/luci.angle.ci/win-msvc-dbg-compile"
-    category: "angle|win|msvc|x64"
+    category: "compile|win|msvc|x64"
     short_name: "dbg"
   }
   builders {
     name: "buildbucket/luci.angle.ci/win-msvc-x86-compile"
-    category: "angle|win|msvc|x86"
+    category: "compile|win|msvc|x86"
     short_name: "rel"
   }
   builders {
     name: "buildbucket/luci.angle.ci/win-msvc-x86-dbg-compile"
-    category: "angle|win|msvc|x86"
+    category: "compile|win|msvc|x86"
     short_name: "dbg"
   }
   builders {
     name: "buildbucket/luci.angle.ci/win-test"
-    category: "angle|win|clang|x64"
+    category: "test|win|clang|x64"
     short_name: "rel"
   }
   builders {
     name: "buildbucket/luci.angle.ci/win-x86-dbg-compile"
-    category: "angle|win|clang|x86"
+    category: "compile|win|clang|x86"
     short_name: "dbg"
   }
   builders {
     name: "buildbucket/luci.angle.ci/win-x86-test"
-    category: "angle|win|clang|x86"
+    category: "test|win|clang|x86"
     short_name: "rel"
   }
   builders {
     name: "buildbucket/luci.angle.ci/winuwp-compile"
-    category: "angle|winuwp|msvc|x64"
+    category: "compile|winuwp|msvc|x64"
     short_name: "rel"
   }
   builders {
     name: "buildbucket/luci.angle.ci/winuwp-dbg-compile"
-    category: "angle|winuwp|msvc|x64"
+    category: "compile|winuwp|msvc|x64"
     short_name: "dbg"
   }
   builders {

infra/config/main.star

@@ -208,14 +208,26 @@ def angle_builder(name, cpu):
     is_uwp = "winuwp" in name
     is_msvc = is_uwp or "-msvc" in name
 
+    location_regexp = None
+
     if name.endswith("-compile"):
         test_mode = "compile_only"
+        category = "compile"
     elif name.endswith("-test"):
         test_mode = "compile_and_test"
+        category = "test"
     elif is_trace:
         test_mode = "trace_tests"
+        category = "trace"
+
+        # Trace tests are only run on CQ if files in the capture folders change.
+        location_regexp = [
+            ".+/[+]/src/libANGLE/capture/.+",
+            ".+/[+]/src/tests/capture.+",
+        ]
     elif is_perf:
         test_mode = "compile_and_test"
+        category = "perf"
     else:
         print("Test mode unknown for %s" % name)
 
@@ -224,6 +236,18 @@ def angle_builder(name, cpu):
     else:
         toolchain = "clang"
 
+    if is_uwp:
+        os_name = "winuwp"
+    else:
+        os_name = config_os.console_name
+
+    if is_perf:
+        short_name = get_gpu_type_from_builder_name(name)
+    elif is_debug:
+        short_name = "dbg"
+    else:
+        short_name = "rel"
+
     properties = {
         "builder_group": "angle",
         "$build/goma": goma_props,
@@ -248,35 +272,10 @@ def angle_builder(name, cpu):
         ),
     )
 
-    # Trace tests are only included automatically if files in the capture folder change.
-    if is_trace:
-        config = "trace"
-        location_regexp = [
-            ".+/[+]/src/libANGLE/capture/.+",
-            ".+/[+]/src/tests/capture.+",
-        ]
-    elif is_perf:
-        config = "perf"
-    else:
-        config = "angle"
-        location_regexp = None
-
-    if is_uwp:
-        os_name = "winuwp"
-    else:
-        os_name = config_os.console_name
-
-    if is_perf:
-        short_name = get_gpu_type_from_builder_name(name)
-    elif is_debug:
-        short_name = "dbg"
-    else:
-        short_name = "rel"
-
     luci.console_view_entry(
         console_view = "ci",
         builder = "ci/" + name,
-        category = config + "|" + os_name + "|" + toolchain + "|" + cpu,
+        category = category + "|" + os_name + "|" + toolchain + "|" + cpu,
         short_name = short_name,
     )