From: Bryan C. Mills <bcmills@google.com>
Date: Tue, 23 Nov 2021 15:54:49 +0000 (-0500)
Subject: misc/reboot: skip TestRepeatBootstrap on short builders
X-Git-Tag: go1.18beta1~202
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=b90c6b99b3fbe60f4782c3e3b85f0ba9bbcf5f50;p=gostls13.git

misc/reboot: skip TestRepeatBootstrap on short builders

This test is slow and resource-intensive, and will rarely catch
failures. It is important to run sometimes, but probably a waste of
time on smaller (and especially reverse) builders.
Rather than hard-coding a list of small builders, only run it on the
longtest builders.

Fixes #35233
Fixes #30892
Fixes #49753

Change-Id: I25a9702e1f541246ea200fd7c79414ca5f69edae
Reviewed-on: https://go-review.googlesource.com/c/go/+/366538
Trust: Bryan C. Mills <bcmills@google.com>
Run-TryBot: Bryan C. Mills <bcmills@google.com>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
---

diff --git a/misc/reboot/reboot_test.go b/misc/reboot/reboot_test.go
index 6bafc608b5..ef164d3232 100644
--- a/misc/reboot/reboot_test.go
+++ b/misc/reboot/reboot_test.go
@@ -15,6 +15,10 @@ import (
 )
 
 func TestRepeatBootstrap(t *testing.T) {
+	if testing.Short() {
+		t.Skipf("skipping test that rebuilds the entire toolchain")
+	}
+
 	goroot, err := os.MkdirTemp("", "reboot-goroot")
 	if err != nil {
 		t.Fatal(err)