Skip to content

Commit 3b519a6

Browse files
committed
dashboard: create misc-compile-ios builder
Create the builder as a post-submit builder with a non-zero KnownIssue value first. Once we know it works well, we'll have the confidence to be able to safely promote it to a normal TryBot, without the risk of causing disruption for contributors. For golang/go#41610. Change-Id: If871addf325b7728109afbe094b72d40e27ceec1 Reviewed-on: https://go-review.googlesource.com/c/build/+/257617 Trust: Dmitri Shuralyov <[email protected]> Run-TryBot: Dmitri Shuralyov <[email protected]> TryBot-Result: Go Bot <[email protected]> Reviewed-by: Andrew Bonventre <[email protected]>
1 parent 711ee32 commit 3b519a6

File tree

2 files changed

+20
-3
lines changed

2 files changed

+20
-3
lines changed

dashboard/builders.go

Lines changed: 20 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -715,8 +715,8 @@ type BuildConfig struct {
715715
HostType string
716716

717717
// KnownIssue is a non-zero golang.org/issue/nnn number for a builder
718-
// that is known to be failing for some reason, such as because it is
719-
// a new builder still in development/testing, or because the feature
718+
// that may fail due to a known issue, such as because it is a new
719+
// builder still in development/testing, or because the feature
720720
// or port that it's meant to test hasn't been added yet, etc.
721721
//
722722
// A non-zero value here means that failures on this builder should not
@@ -1560,6 +1560,24 @@ func init() {
15601560
addMiscCompile("-netbsd", "^netbsd-") // 4: amd64, 386, arm, arm64
15611561
addMiscCompile("-openbsd", "^openbsd-") // 4: amd64, 386, arm, arm64
15621562

1563+
// TODO(golang.org/issue/41610): Test misc-compile-ios as a post-
1564+
// submit builder with a known issue, before promoting to trybot.
1565+
func(suffix, rx string) {
1566+
addBuilder(BuildConfig{
1567+
Name: "misc-compile" + suffix,
1568+
HostType: "host-linux-jessie",
1569+
buildsRepo: func(repo, branch, goBranch string) bool { return repo == "go" && branch == "master" },
1570+
KnownIssue: 41610,
1571+
env: []string{"GO_DISABLE_OUTBOUND_NETWORK=1"},
1572+
CompileOnly: true,
1573+
Notes: "Runs buildall.sh to cross-compile & vet std+cmd packages for " + rx + ", but doesn't run any tests.",
1574+
allScriptArgs: []string{
1575+
// Filtering pattern to buildall.bash:
1576+
rx,
1577+
},
1578+
})
1579+
}("-ios", "^ios-") // 1: arm64 (for Go 1.16 and newer)
1580+
15631581
// And 3 that don't fit above:
15641582
addMiscCompile("-other", "^(linux-s390x|linux-riscv64|dragonfly-amd64)$")
15651583
// TODO: Issue 25963, get the misc-compile trybots for

dashboard/builders_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -790,7 +790,6 @@ func TestTryBotsCompileAllPorts(t *testing.T) {
790790
done["nacl-386"] = true // removed in Go 1.14
791791
done["nacl-arm"] = true // removed in Go 1.14
792792
done["windows-arm"] = true // TODO(golang.org/issue/38607) disabled until builder is replaced
793-
done["ios-arm64"] = true // TODO(golang.org/issue/41610): Add builder for ios/arm64.
794793
check := func(goos, goarch string) {
795794
if goos == "android" {
796795
// TODO(golang.org/issue/25963): support

0 commit comments

Comments
 (0)