This is a re-send of CL 216765, which was reverted in CL 217723
due to the misc-compile-other trybot being broken on linux/riscv64.
By now, the problem that was caught by the misc-compile-other trybot
has been fixed in CL 217777, so we can try again.
Fixesgolang/go#36871Fixesgolang/go#37022
Change-Id: Id9e7d7f3e610b17a2d37a2216ab5d3eed139ee1e
Reviewed-on: https://go-review.googlesource.com/c/build/+/224580
Reviewed-by: Alexander Rakoczy <alex@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
If a dev branch is active, it will be current. If it is not
active, it is probably not current, but we don't build on the
branch so it doesn't matter anyway.
Specifically, the dev.link branch is currently under active
development, and it would be good to have a large set of builders
run on the branch to catch problems. (E.g., AIX build was broken
but not noticed.)
Fixesgolang/go#37953.
Change-Id: Idd7e3edaaff1b4d2cee17da552a96df2b00716e9
Reviewed-on: https://go-review.googlesource.com/c/build/+/223381
Run-TryBot: Cherry Zhang <cherryyz@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
This is a new linux/amd64 GCE builder for the purpose of
testing the GOEXPERIMENT=staticlockranking configuration
which is landing to master branch soon.
It runs only on the main Go repository master branch and
release branches 1.15 and newer. No golang.org/x repos.
Not included in trybots by default, but can be requested
via SlowBots by saying TRY=linux-amd64-staticlockranking.
Also expand documentation of BuildConfig.buildsRepo field
to make it more clear how to use it. A common mistake for
custom buildsRepo functions that are added is to create a
custom policy with a minor adjustment and forgetting to
preserve the rest of the default build repo policy.
Fixesgolang/go#37937
Change-Id: Ic56f82e9ede30cb6fe8f6cc7d0282d6a0a12c1b8
Reviewed-on: https://go-review.googlesource.com/c/build/+/224078
Reviewed-by: Robert Findley <rfindley@google.com>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
Run-TryBot: Alexander Rakoczy <alex@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Go 1.15 will require macOS 10.12 and newer¹, so keep the OS X 10.11
builder enabled only on release branches for 1.14 and older.
¹ https://golang.org/doc/go1.14#darwinFixesgolang/go#37425
Change-Id: I89ab0eb56729fe16fb3a524c8f7b9bb25751d708
Reviewed-on: https://go-review.googlesource.com/c/build/+/222797
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
These have been consistently failing for weeks and attempts to
contact the owner have been unsuccessful.
Updates golang/go#37611Fixesgolang/go#31497
Change-Id: I43c8b9de18c758d256290ee42eb3755dd9eacf3f
Reviewed-on: https://go-review.googlesource.com/c/build/+/222720
Run-TryBot: Andrew Bonventre <andybons@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
OS X 10.10 is no longer supported by the latest versions
of Go (1.13 and 1.14). It is no longer in use.
Fixesgolang/go#37713
Change-Id: Ibc4a527c17f2dafd00ee0fba2e312394e1ceac9c
Reviewed-on: https://go-review.googlesource.com/c/build/+/222337
Run-TryBot: Carlos Amedee <carlos@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
A kernel bug in macOS 10.14 is causing a set of tests to be flaky.
Upgrading to macOS 10.15 fixes this issue. This change moves more
testing towards the current version of macOS.
Fixesgolang/go#37695
Change-Id: Ide69693e760639a9d680738cad0541d79a0a0443
Reviewed-on: https://go-review.googlesource.com/c/build/+/222238
Run-TryBot: Carlos Amedee <carlos@golang.org>
Reviewed-by: Bryan C. Mills <bcmills@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
When we moved the ppc64le builders to Docker, GOPPC64=power9 was not set
for the POWER9 builders. This variable has to be passed to the environment
so the compiler can generate POWER9 instructions.
Fixesgolang/go#36802
Change-Id: I13e99eb316e2126302e26ee771aaf8bbfbbc1c8c
Reviewed-on: https://go-review.googlesource.com/c/build/+/216577
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
macOS 10.8 is very old and not supported by Go 1.13 nor Go 1.12.
There are no instances of this builder type available because
macOS builders are not created on demand and the ExpectNum value
is set to zero. Remove it and various files related to it.
Fixesgolang/go#36560
Change-Id: Ife36624f3695fccff2f5a12af4518e25645b4f0d
Reviewed-on: https://go-review.googlesource.com/c/build/+/215257
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Start using n1-highcpu-8 machine type instead of n1-highcpu-4
for the freebsd-amd64-race builder.
The freebsd-amd64-race builder has produced good test results
for the x/tools repo for a long time, but by now it has started
to consistently fail for reasons that seem connected to it having
only 3.6 GB memory. The Windows race builders needed to be bumped
from 7.2 GB to 14.4 GB to run successfully, so this change makes
a small incremental step to bring freebsd-amd64-race closer in
line with other builders. If memory-related problems continue to
occur with 7.2 GB, the next step will be to go up to 14.4 GB.
The freebsd-amd64-race builder is using an older version of FreeBSD.
We may want to start using a newer one for running tests with -race,
but that should be a separate change so we can see the results of
this change without another confounding variable.
Also update all FreeBSD builders to use https in buildletURLTmpl,
because it's expected to work fine and will be more consistent.
Updates golang/go#36444
Updates golang/go#34621
Updates golang/go#29252
Updates golang/go#33986
Change-Id: Idfcefd1c91bddc9f70ab23e02fcdca54fda9d1ac
Reviewed-on: https://go-review.googlesource.com/c/build/+/214433
Run-TryBot: Carlos Amedee <carlos@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
It was unintentionally left out, causing these builders
to not skip x/mobile on Go 1.12.
Fixesgolang/go#36506
Change-Id: I19a9e2838c8bdaf68fe21151ea9454d1f3062032
Reviewed-on: https://go-review.googlesource.com/c/build/+/214294
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
We do not support Go 1.11 by now, so there is no longer a need to
include filtering for release-branch.go1.11. Remove it to simplify
code and remove useless test cases.
There was a bug where the buildsRepo policy for freebsd-386-10_3,
freebsd-386-10_4, and freebsd-386-11_1 builders forgot to include
the default build policy, so it was testing x/mobile unintentionally.
Add a test case for its current behavior. It'll be fixed in next CL.
Updates golang/go#36506
Change-Id: I4b2ff6cae417bf4699ec99f5c427b8531e12e915
Reviewed-on: https://go-review.googlesource.com/c/build/+/214293
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
Start using n1-highcpu-16 machine type instead of n1-highcpu-8
for windows-amd64-longtest and windows-amd64-race builders.
The primary motivation of this change is to increase amount of
RAM available on the race builder from 7.2 GB to 14.4 GB.
CL 208500 has helped by going from 3.6 to 7.2, but empirically
8 GB is not enough for all tests to pass, while 16 GB can be.
We already use 16 GB for some other race and/or long builders.
Updates golang/go#35186
Updates golang/go#33951
Updates golang/go#33986
Change-Id: I8031a517e6e44ce5e5faba085e7c781a1fcab92e
Reviewed-on: https://go-review.googlesource.com/c/build/+/213557
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
The packet arm64 build box has 96 cores, and easily gets overloaded with
multiple builds running simultaneously. Restrict each container to fewer
cores, and decrease the number of workers.
In order to not lose some of the coverage we got from packet as a
high-CPU builder, we will need to create new builder types going forward
to ensure we don't lose coverage. See the issue for details.
Updates golang/go#36170
Change-Id: I16478fa05258bc14ac59a1e708133ffcc4c5a664
Reviewed-on: https://go-review.googlesource.com/c/build/+/211579
Run-TryBot: Alexander Rakoczy <alex@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
This partially reverts CL 192327.
Reason for revert: x/review failure turned out to be a bug in the test.
Updates golang/go#32836
Change-Id: I189ad4ebc1b48fbdbb64f3e42ab1459761ff0c8c
Reviewed-on: https://go-review.googlesource.com/c/build/+/211497
Run-TryBot: Bryan C. Mills <bcmills@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
They used to run on Kubernetes (and still could) but they're just a
container and we run them on COS nowadays.
Change-Id: I21a6db90efc5071997c6d6dcae1a42a7c28afa3a
Reviewed-on: https://go-review.googlesource.com/c/build/+/209957
Reviewed-by: Alexander Rakoczy <alex@golang.org>
Run-TryBot: Alexander Rakoczy <alex@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Start using n1-highcpu-8 machine type instead of n1-highcpu-4
for windows-amd64-longtest and windows-amd64-race builders.
Continue to use the existing n1-highcpu-4 machine type for all
other windows/386 and windows/amd64 builders for now.
The primary motivation of this change is to increase amount of
RAM available on the longtest builder from 3.6 GB to 7.2 GB,
because it is consistenly failing due to being out of memory.
Also fix minor typos, and increase consistency in comments and notes.
Updates golang/go#33951
Change-Id: Iac5da67fa584059842850a9c8deda98c242741f2
Reviewed-on: https://go-review.googlesource.com/c/build/+/208500
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
This change enables snapshots on Plan 9 builders,
so running the tests for each of the golang.org/x
repositories will not require to rebuild Go every time.
Change-Id: I305b713a48c155ebba2c03f39633d17a6fd71e6f
Reviewed-on: https://go-review.googlesource.com/c/build/+/208359
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Added the macOS 10.15 Catalina builder to the dashboard. Catalina builders should
use the Mojave guestid in vSphere 6.7 update 3 because the Catalina guestid is not
available in this version of vSphere. A Catalina builder will run on this version
of vSphere with the Mojave guestid. Modified the expected number of vm instances
running on Macstadium due to the addition of macOS 10.15 builders.
Updates golang/go#34748
Change-Id: If14f6e088afc05dd1783460f21907e9345e3c338
Reviewed-on: https://go-review.googlesource.com/c/build/+/207937
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Optimizations and tuning remain, but this should be tons better than
what we had before (random).
Updates golang/go#19178
Change-Id: Idb483a4c4209a012814322cc8b37b966ee4681de
Reviewed-on: https://go-review.googlesource.com/c/build/+/205078
Reviewed-by: Bryan C. Mills <bcmills@google.com>
Currently, android/{386,arm,arm64} are not covered, so add a
misc-compile builder for these.
Also correct a typo in the comment for the darwin misc-compile builder
and add update the misc-compile-freebsd comment to include
freebsd/arm64.
Change-Id: Ica3b27e98db73c10edf30260acddcd7ea57d9051
Reviewed-on: https://go-review.googlesource.com/c/build/+/205798
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Emmanuel Odeke <emm.odeke@gmail.com>
All the buildlets have been updated to use --reverse-type with a host
type instead of a builder type.
Fixesgolang/go#21260
Change-Id: I1264261f099c3686fe01455949486f523b94c6de
Reviewed-on: https://go-review.googlesource.com/c/build/+/205608
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Bryan C. Mills <bcmills@google.com>
This will allow the openbsd-arm builder to be migrated to a buildlet.
Change-Id: If38788daf2d1e7f237ebdcce384a098e08bffb0b
Reviewed-on: https://go-review.googlesource.com/c/build/+/192598
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
From https://github.com/golang/go/issues/34958#issuecomment-543852995 :
> Go's DragonFly support policy is that we support the latest stable
> release primarily, but also try to keep DragonFly master passing, in
> prep for it to become the latest stable release.
>
> But that does mean we need one more builder at the moment.
Updates golang/go#34958
Updates golang/go#23060
Change-Id: I84be7c64eac593dee2252c397f9529deea13605a
Reviewed-on: https://go-review.googlesource.com/c/build/+/202478
Reviewed-by: Tobias Klauser <tobias.klauser@gmail.com>
Reviewed-by: Bryan C. Mills <bcmills@google.com>
Dmitri Shuralyov
Cherry Zhang
Andrew Bonventre
Carlos Amedee
Brad Fitzpatrick
Alexander Rakoczy
Carlos Eduardo Seo
Bryan C. Mills
Tobias Klauser
David du Colombier
Joel Sing
Dmitri Goutnik