main.star: include longtest and race in pre-submit on release branches

Given release branches only get backports and true development should
be happening on the main branch, release branches prioritize catching
more problems before submission over pre-submit completion speed, and
so they choose not to skip longtest and race builders.

This is a part of the current trybot policy implemented in x/build,
and seems to be generally working well.

For golang/go#37827.

Change-Id: I1c9bcf600a7ea5da32296a8ddfcda083da839c29
Reviewed-on: https://go-review.googlesource.com/c/build/+/522175
Reviewed-by: Heschi Kreinick <[email protected]>
TryBot-Bypass: Dmitri Shuralyov <[email protected]>
Auto-Submit: Dmitri Shuralyov <[email protected]>
Reviewed-by: Dmitri Shuralyov <[email protected]>

Author: dmitshur

generated/commit-queue.cfg

@@ -6700,7 +6700,6 @@ config_groups {
       }
       builders {
         name: "golang/security-try/go1.20-linux-386-longtest"
-        includable_only: true
       }
       builders {
         name: "golang/security-try/go1.20-linux-amd64"
@@ -6710,18 +6709,15 @@ config_groups {
       }
       builders {
         name: "golang/security-try/go1.20-linux-amd64-longtest"
-        includable_only: true
       }
       builders {
         name: "golang/security-try/go1.20-linux-amd64-longtest-race"
-        includable_only: true
       }
       builders {
         name: "golang/security-try/go1.20-linux-amd64-misccompile"
       }
       builders {
         name: "golang/security-try/go1.20-linux-amd64-race"
-        includable_only: true
       }
       builders {
         name: "golang/security-try/go1.20-linux-arm64"
@@ -6738,11 +6734,9 @@ config_groups {
       }
       builders {
         name: "golang/security-try/go1.20-windows-amd64-longtest"
-        includable_only: true
       }
       builders {
         name: "golang/security-try/go1.20-windows-amd64-race"
-        includable_only: true
       }
       builders {
         name: "golang/try/tricium"
@@ -7045,7 +7039,6 @@ config_groups {
       }
       builders {
         name: "golang/security-try/go1.21-linux-386-longtest"
-        includable_only: true
       }
       builders {
         name: "golang/security-try/go1.21-linux-amd64"
@@ -7055,18 +7048,15 @@ config_groups {
       }
       builders {
         name: "golang/security-try/go1.21-linux-amd64-longtest"
-        includable_only: true
       }
       builders {
         name: "golang/security-try/go1.21-linux-amd64-longtest-race"
-        includable_only: true
       }
       builders {
         name: "golang/security-try/go1.21-linux-amd64-misccompile"
       }
       builders {
         name: "golang/security-try/go1.21-linux-amd64-race"
-        includable_only: true
       }
       builders {
         name: "golang/security-try/go1.21-linux-arm64"
@@ -7083,11 +7073,9 @@ config_groups {
       }
       builders {
         name: "golang/security-try/go1.21-windows-amd64-longtest"
-        includable_only: true
       }
       builders {
         name: "golang/security-try/go1.21-windows-amd64-race"
-        includable_only: true
       }
       builders {
         name: "golang/try/tricium"
@@ -7733,7 +7721,6 @@ config_groups {
       }
       builders {
         name: "golang/try/go1.20-linux-386-longtest"
-        includable_only: true
       }
       builders {
         name: "golang/try/go1.20-linux-amd64"
@@ -7743,18 +7730,15 @@ config_groups {
       }
       builders {
         name: "golang/try/go1.20-linux-amd64-longtest"
-        includable_only: true
       }
       builders {
         name: "golang/try/go1.20-linux-amd64-longtest-race"
-        includable_only: true
       }
       builders {
         name: "golang/try/go1.20-linux-amd64-misccompile"
       }
       builders {
         name: "golang/try/go1.20-linux-amd64-race"
-        includable_only: true
       }
       builders {
         name: "golang/try/go1.20-linux-arm64"
@@ -7771,11 +7755,9 @@ config_groups {
       }
       builders {
         name: "golang/try/go1.20-windows-amd64-longtest"
-        includable_only: true
       }
       builders {
         name: "golang/try/go1.20-windows-amd64-race"
-        includable_only: true
       }
       builders {
         name: "golang/try/tricium"
@@ -8103,7 +8085,6 @@ config_groups {
       }
       builders {
         name: "golang/try/go1.21-linux-386-longtest"
-        includable_only: true
       }
       builders {
         name: "golang/try/go1.21-linux-amd64"
@@ -8113,18 +8094,15 @@ config_groups {
       }
       builders {
         name: "golang/try/go1.21-linux-amd64-longtest"
-        includable_only: true
       }
       builders {
         name: "golang/try/go1.21-linux-amd64-longtest-race"
-        includable_only: true
       }
       builders {
         name: "golang/try/go1.21-linux-amd64-misccompile"
       }
       builders {
         name: "golang/try/go1.21-linux-amd64-race"
-        includable_only: true
       }
       builders {
         name: "golang/try/go1.21-linux-arm64"
@@ -8141,11 +8119,9 @@ config_groups {
       }
       builders {
         name: "golang/try/go1.21-windows-amd64-longtest"
-        includable_only: true
       }
       builders {
         name: "golang/try/go1.21-windows-amd64-race"
-        includable_only: true
       }
       builders {
         name: "golang/try/tricium"

main.star

@@ -632,6 +632,10 @@ def enabled(low_capacity_hosts, project, go_branch_short, builder_type):
     pt = PROJECTS[project]
     os, arch, _, run_mods = split_builder_type(builder_type)
 
+    presubmit_skip_run_mods = ["longtest", "race"]
+    if project == "go" and go_branch_short != "gotip":
+        presubmit_skip_run_mods = [] # See go.dev/issue/37827.
+
     enable_types = None
     if pt == PT.TOOL:
         enable_types = ["linux-amd64", "windows-amd64", "darwin-amd64"]
@@ -641,7 +645,7 @@ def enabled(low_capacity_hosts, project, go_branch_short, builder_type):
         fail("unhandled SPECIAL project: %s" % project)
     postsubmit = enable_types == None or any([x == "%s-%s" % (os, arch) for x in enable_types])
     presubmit = postsubmit \
-        and not any([x in run_mods for x in ["longtest", "race"]]) \
+        and not any([x in run_mods for x in presubmit_skip_run_mods]) \
         and not is_capacity_constrained(low_capacity_hosts, builder_type)
     return presubmit, postsubmit