The wasmtime WASI runtime completes the standard
library tests faster than wazero, which
makes it suitable for running on every CL.
Wazero will continue to be run against tip.
Querying the builders database shows the following statistics
for builder runtime (execution in seconds):
wasip1-wasm-wasmtime:
1234.808416431
852.305792789
1252.596007033
wasip1-wasm-wazero:
2421.876266138
2419.399971708
2439.044948166
For golang/go#59150.
Change-Id: I68c7d454109e53e860a020aabd489c6fb97121dc
Reviewed-on: https://go-review.googlesource.com/c/build/+/498675
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Reviewed-by: Eli Bendersky <eliben@google.com>
The wasmtime WASI runtime now passes the standard library test suite.
For golang/go#59583.
Change-Id: I8e87cb46f3b7799996d4f1312478885e22fca1f8
Reviewed-on: https://go-review.googlesource.com/c/build/+/498655
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Reviewed-by: Achille Roussel <achille.roussel@gmail.com>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Run-TryBot: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
The coordinator has some custom logic for "go_test:*" dist test names
involved in partitioning, merging, and sorting them using historical
test timing data. We also have many builder-specific dist test adjust
policies, and some parts of them have become out of date (e.g.,
macTestPolicy skips "wiki", a dist test that's long gone).
Go 1.21 is simplifying the naming pattern for dist test names, and
we want to do that without having to manually update the (many!)
existing adjust policies, as well as to avoid breaking or creating
a need to re-engineer the existing mechanisms that make "go_test:*"
dist test names special.
Make that possible by remapping the 'go tool dist test -list' output
from the new "pkg[:variant]" format to the previous format (or pass it
through unmodified if it's already in the old format) for the purpose
of dist test adjust policies only. Also keep the raw unmodified dist
test name around since it's needed for invoking said dist tests.
For golang/go#37486.
For golang/go#59990.
Change-Id: Ie958609de2a810fa0053a00091a0c54c3f8bfd83
Reviewed-on: https://go-review.googlesource.com/c/build/+/496190
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Austin Clements <austin@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
This reverts commit e0d0a8242f.
Reason for revert: We want to keep all builders active for x/telemetry. We can add workarounds instead.
Change-Id: Ib3bed9de818c6424f4d62bc0b00aa8e7a1776b6d
Reviewed-on: https://go-review.googlesource.com/c/build/+/495936
Reviewed-by: Heschi Kreinick <heschi@google.com>
Auto-Submit: Jamal Carvalho <jamal@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Jamal Carvalho <jamal@golang.org>
- The x/telemetry repo depends on slog which requires at least go1.20.
- A dependency is not supported on plan9.
Change-Id: Ia2e911c160b9aa33e91a28133fd21bf65a754d53
Reviewed-on: https://go-review.googlesource.com/c/build/+/495760
Run-TryBot: Jamal Carvalho <jamal@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
This tries to address intermittent test failures by setting
the GO_TEST_TIMEOUT_SCALE=2 for this builder.
Fixesgolang/go#60109.
Change-Id: If980795997410ec6d4e927a105d5fc9f1fd8edae
Reviewed-on: https://go-review.googlesource.com/c/build/+/495655
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
The wasmedge runner provides the wasmedge.org WASI runtime
for testing the wasip1 implementation.
For golang/go#60097
Change-Id: I1148977296d00163f638d4fcc7a916c58c6d17c5
Reviewed-on: https://go-review.googlesource.com/c/build/+/493696
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
Auto-Submit: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Run-TryBot: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
The wasmer runner privdes the wasmer.io WASI runtime
for testing the wasip1 implementation.
For golang/go#59907
Change-Id: Ied892dd60d933666b53039ab1c9d3e3065c8800b
Reviewed-on: https://go-review.googlesource.com/c/build/+/493755
Reviewed-by: Cherry Mui <cherryyz@google.com>
Run-TryBot: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
An upcoming change (CL 492776) adds a dependency that doesn't build on
plan9.
For golang/go#59672.
Change-Id: If1dbb99e859ecef406aaaf754745d5ce95aa7037
Reviewed-on: https://go-review.googlesource.com/c/build/+/493624
Run-TryBot: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Michael Knyszek <mknyszek@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
This is a new linux/s390x builder for race detection. It
runs only on the main Go repository master branch, not on golang.org/x repos.
Fixesgolang/go#59962
Change-Id: I1149b2ccf51ca020ba4ea489f189825255a85e2e
Reviewed-on: https://go-review.googlesource.com/c/build/+/492577
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Cherry Mui <cherryyz@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Michael Pratt <mpratt@google.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
The darwin longtest builder was added during the Go 1.21 dev cycle and
needed some development work before it would pass all tests. Disable it
on older release branches that don't have said work, and make it run on
Go 1.21 and newer release branches as a TryBot like all other longtest
builders.
This makes it consistent with other -longtest builders and will help
avoid entering broken state when backporting to future release branches.
Also remove redundant version check from windows longtest tryBot policy
functions since the version check is already there in buildsRepo policy.
For golang/go#35678.
For golang/go#37827.
Change-Id: I09c479f1be2b3635c385b504b0c86b50e65e696b
Reviewed-on: https://go-review.googlesource.com/c/build/+/492055
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Michael Pratt <mpratt@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
These builders seem to benefit greatly from CL 451219, and it doesn't
seem worth applying workarounds in x-repo tests in order to support
the builders on 1.19.
For golang/go#59769.
Change-Id: Ie82e4c8904b0c3cb31a4aa9bb9a3a768c56b3107
Reviewed-on: https://go-review.googlesource.com/c/build/+/489456
Run-TryBot: Bryan Mills <bcmills@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Bryan Mills <bcmills@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
The new GOWASIRUNTIME variable will be used to select the WASI runtime
in the executable script.
For golang/go#59583
Change-Id: Ie2d5f9373393b4c40ba592f956e5ad1e4b1fd207
Reviewed-on: https://go-review.googlesource.com/c/build/+/485556
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Auto-Submit: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Run-TryBot: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Now that we're cross-compiling everything, we can build all ports
(except ios/android/js, of course). Add a script that uses the gotip
command to populate a list of ports for the latest development version.
Change-Id: I7afdeef739934a8dee79cec033f1782b661cb848
Reviewed-on: https://go-review.googlesource.com/c/build/+/479278
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Heschi Kreinick <heschi@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Carlos Amedee <carlos@golang.org>
Run-TryBot: Heschi Kreinick <heschi@google.com>
Also make vulndb run linux-amd64-race.
(Everyone should run the race detector!)
Also make mobile run on linux-amd64 trybot.
It already runs on linux-amd64 post-commit.
Also make mobile run on linux-amd64-race trybot.
(Everyone should run the race detector!)
Also run linux-amd64-longtest-race post-commit
on every repo, just like linux-amd64-longtest does.
Change-Id: I959d8cc2128ae4050b63451ed1e1cf047df172a8
Reviewed-on: https://go-review.googlesource.com/c/build/+/480935
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
Run-TryBot: Russ Cox <rsc@golang.org>
Reviewed-by: Bryan Mills <bcmills@google.com>
Reviewed-by: David Chase <drchase@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
This VM hosted by osuol allows CI testing of power10 big-endian.
For golang/go#59614
Change-Id: Ife151a984c4e8fa0e4b9e6e459b10f1cf485a765
Reviewed-on: https://go-review.googlesource.com/c/build/+/484175
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: David Chase <drchase@google.com>
Mark it as broken until the port is complete.
Updates golang/go#56001
Change-Id: Ic8544efcd814b217811c1a8ccccdc572365216b7
Reviewed-on: https://go-review.googlesource.com/c/build/+/484355
Reviewed-by: David Chase <drchase@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
The initial issue has been fixed, but a separate issue has been
created to track the wasmtime specific problems. Update the
issue tracked in the builder dashboard.
Change-Id: I3f011b9a9ccf29971a0ab7fb37e1e2c3aba1997f
Reviewed-on: https://go-review.googlesource.com/c/build/+/483956
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Reviewed-by: David Chase <drchase@google.com>
Auto-Submit: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Now that the wasip1 port has been fully merged, we should
enable the wazero builder on all commits to master and the
1.21 release branch.
For golang/go#58141
Change-Id: I112b854812d0b4f4d628f203a1ff271d310a57b8
Reviewed-on: https://go-review.googlesource.com/c/build/+/483817
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Auto-Submit: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
The existing behavior for setting GOPROXY is rather hard to follow, and
doesn't work correctly in many cases. For example, longtest on a reverse
builder gets the GKE proxy. Before CL 479837 there were no longtest
builders outside of GCE, so this case was never covered. Fixing this is
the motivation of this CL.
They way configuration works today is:
1. buildstatus.go unconditionally sets GOPROXY to the GKE proxy [1].
2. st.conf.ModuleEnv potentially overrides GOPROXY with a more
reasonable setting, with a bunch of complex conditions.
Unify and simplify this process by moving it into buildstatus.go, where
their is now a strict ordering of possible GOPROXY values. Notable
changes:
* The GKE proxy is never used outside of GCE.
* There is a consistent default/fallback of proxy.golang.org.
I initially tried to split this into two CLs: one unifying the
implementation and the next changing the behavior, but the old behavior
is so mind-boggling that the first CL doesn't really make much sense.
The annoying part of this CL is that tests move from dashboard to
cmd/coordinator, requiring us to export additional fields so
cmd/coordinator tests can configure the builders.
The test cases themselves are unchanged except for the addition of a
non-GCE longtest builder case.
[1] Except in runSubrepoTests, which avoids doing so for reverse
builders. This was a workaround for private proxy builders in CL 275412,
but wasn't extended to other callers because only subrepo tests were
seeing a regression. More strangeness.
For golang/go#35678.
Change-Id: I6090c8c5e91ce6be9bfc07c16f36ed339c9d27ae
Reviewed-on: https://go-review.googlesource.com/c/build/+/482339
Reviewed-by: Bryan Mills <bcmills@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
Run-TryBot: Michael Pratt <mpratt@google.com>
Perhaps it'll be helpful to have one. It picks one of the
two boringcrypto builders for you, and is shorter to type.
Change-Id: I55a6ab921b02f11d25fdc20be7fa0598a559da01
Reviewed-on: https://go-review.googlesource.com/c/build/+/482336
Reviewed-by: Ian Lance Taylor <iant@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
This field was added in CL 166218 but never used.
For golang/go#35678.
Change-Id: I197fd3835136e7649b20a7ca47d4179591cb49a5
Reviewed-on: https://go-review.googlesource.com/c/build/+/482335
Auto-Submit: Michael Pratt <mpratt@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Michael Pratt <mpratt@google.com>
This change tries to avoid suspected kernel issues with COS release 105
that break ASAN and MSAN for both clang and gcc.
Updates #59419.
Updates #59418.
Change-Id: I7ae3768d106ea9152bae6d215640b384a7e68e05
Reviewed-on: https://go-review.googlesource.com/c/build/+/482195
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Run-TryBot: Michael Knyszek <mknyszek@google.com>
Post-submit builds the same set as other "secondary" longtest builders,
like linux-386-longtest.
Trybot builds are not enabled for release branches due to lack of
capacity.
For golang/go#35678.
Change-Id: I60f15bfe8fe74072f35b20462cd9d13a111a1b31
Reviewed-on: https://go-review.googlesource.com/c/build/+/479837
Reviewed-by: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Michael Pratt <mpratt@google.com>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Run-TryBot: Michael Pratt <mpratt@google.com>
Update the "windows-386" slowbot alias to windows-386-2016
from windows-386-2008.
Updates golang/go#58007.
Change-Id: I44e10500fcf0520976a80916b1fc4a42307cd7a3
Reviewed-on: https://go-review.googlesource.com/c/build/+/470715
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Michael Pratt <mpratt@google.com>
Run-TryBot: Than McIntosh <thanm@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
CL 468937 added a newer Windows 32-bit TryBot that will provide coverage
for the windows/386 port for Go 1.21 and newer. Stop the older Windows 8
builders from running on the relevant branches.
Updates golang/go#58007.
Fixesgolang/go#58008.
Change-Id: I8664a74543b2225986586440ab027cda85b67a3e
Reviewed-on: https://go-review.googlesource.com/c/build/+/463579
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
This new builder has minimal flakes, add it to the TryBot set. Note that
CL 463579 will remove windows-386-2012 from tip.
Fixesgolang/go#58007
Change-Id: I1c7e20cb2158f08fe9ead49fb203d92971e0a8cf
Reviewed-on: https://go-review.googlesource.com/c/build/+/468937
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Michael Pratt <mpratt@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Michael Pratt <mpratt@google.com>
This builder is equivalent to windows-amd64-2016, but using
GOARCH/GOHOSTARCH=386.
Older versions of Windows will eventually drop Go support, so we need to
have builders for newer versions.
For golang/go#58007
Change-Id: Ifdbe38aff735c91f1714e82b3e70346dd9e40b9b
Reviewed-on: https://go-review.googlesource.com/c/build/+/468540
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Michael Pratt <mpratt@google.com>
Auto-Submit: Michael Pratt <mpratt@google.com>
Reviewed-by: Than McIntosh <thanm@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
The linux-arm-aws trybots do not reliably complete within the expected
time limit. This change removes it from the trybot set until we can
reliably run it within the allocated time. Until then the misc-compile
builder will provide minimal coverage during trybot runs. The post
submit builds will still run.
Fixesgolang/go#54679
Change-Id: I712ab4a56dd7ce923c43a08b3461d96ba6638947
Reviewed-on: https://go-review.googlesource.com/c/build/+/468535
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Carlos Amedee <carlos@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Run-TryBot: Carlos Amedee <carlos@golang.org>
A patch (CL 358854) implementing the loong64 GNU format disassembler on x/arch
has been uploaded, and the linux/loong64 builders seem to have enough resources
to handle it.
Change-Id: Ibe4d16e544343a4056ff9bd6b8ae999c4b8b6464
Reviewed-on: https://go-review.googlesource.com/c/build/+/465475
Reviewed-by: Carlos Amedee <carlos@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Run-TryBot: Carlos Amedee <carlos@golang.org>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: xiaodong liu <teaofmoli@gmail.com>
I had initially added known issues fairly aggressively in order to use
them to reduce noise in 'greplogs -triage'. Now that we are using
'watchflakes' for triage, that noise reduction is no longer important
(the failures are already clustered to their respective known issues),
and having greyed-out cells on the dashboard makes new regressions too
easy to miss.
Concretely:
- golang/go#42212 is mostly specific to x/net at this point (as
golang/go#57841)
- There have been no failures matching golang/go#51001 since October.
- golang/go#52724 has been so rare lately that we hadn't yet added a
'watchflakes' pattern for it.
- There have been no failures matching golang/go#51443 since May.
- There have been no failures matching golang/go#53116 or
golang/go#53093 since I enabled 'watchflakes' for the builder in
December.
- The linux-amd64-perf builder seems to be passing consistently for
x/benchmarks and x/tools, so there is no need to refer to
golang/go#53538 to explain failures on it.
Change-Id: Ia16db2a23e5fa037a299f1f56fb26f1cf84521e1
Reviewed-on: https://go-review.googlesource.com/c/build/+/465156
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Bryan Mills <bcmills@google.com>
Auto-Submit: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
The previous change added a StopAfterMake requirement so that the
misc-compile builders would only ever run make.bash. However, this
is a fairly big hammer as it skips everything after make.bash. Coming
up, we're going to want to runSubrepoTests.
However, we can't call into runTests because of golang/go#58297, so we
need a special exception for that. This change thus also adds the notion
of IsCrossCompileOnly defined by whether the host arch doesn't line up
with the target arch, and adds a test to make sure this only applies to
the misc-compile builders.
This change also removes some hard-coding of the misc-compile prefix in
favor of this new IsCrossCompileOnly notion.
For golang/go#58163.
Change-Id: I40ce91e13b45e8bbbb607aedd302f0ec0fbd608a
Reviewed-on: https://go-review.googlesource.com/c/build/+/464956
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
This change removes misc-compile builders' dependence on buildall.bash
and switches to make.bash instead, since they now each build for exactly
one platform.
This change then also removes SplitMakeRun and any paths derived from
that in order to simplify the code.
For golang/go#58163.
Change-Id: I16704cc553c0f6e2c6c34994999693b6d44adb88
Reviewed-on: https://go-review.googlesource.com/c/build/+/464955
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Michael Knyszek <mknyszek@google.com>
This change splits up the misc-compile builders to only build for one
platform each. This is the first step in a sequence of CLs to simplify
misc-compile builders by eliminating the !SplitMakeRun paths, of which
misc-compile builders are the only ones.
The end result will be the support of misc-compile builders for
subrepos.
Splitting up misc-compile builders has the potential to increase VM
demand, but each build also takes less time to run. The main way is
could increase overall VM resource usage is the overhead of setting up
and tearing down a VM. The other downside is more visual noise, though
this is fairly minor.
Note that this change also passes GOOS, GOARCH, and GOARM environment
variables to buildall.sh. The goal is to eventually remove buildall.sh
by just doing make.bash. (Right now buildall.sh will essentially ignore
those variables and just set them itself.)
For golang/go#58163.
Change-Id: Ia48f6a16d080e9e078b2959dea3b68fe43def8ec
Reviewed-on: https://go-review.googlesource.com/c/build/+/463777
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Michael Pratt <mpratt@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Run-TryBot: Michael Knyszek <mknyszek@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
After all.bash is passing with Node.js 18, the js/wasm builder can be
enabled as a regular TryBot again (this time with Node 18 in place of
Node 14).
For golang/go#57614.
Change-Id: I714380fba89e37275aed4144a1e8b50e0cc57884
Reviewed-on: https://go-review.googlesource.com/c/build/+/464036
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Carlos Amedee <carlos@golang.org>
The js/wasm port maintainers want to start landing changes for
Go 1.21 that replace Node.js 14 support with Node.js 18 support.
Disable the js-wasm TryBot on tip so that work can begin.
Once done, the js-wasm-node18 builder can become the new TryBot
on tip, and the temporary misc-compile trybot can be removed.
For golang/go#57614.
Change-Id: I18ebbc69c8da2b7296dd06a5a10c8557395d600d
Reviewed-on: https://go-review.googlesource.com/c/build/+/463915
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Johan Brandhorst-Satzkorn <johan.brandhorst@gmail.com>
Reviewed-by: Carlos Amedee <carlos@golang.org>
openbsd-mips64 does not build anymore with the current x/sys,
an intentional decision by the openbsd mips64 maintainers.
Remove it from our build set to ignore that breakage for now.
The maintainers are welcome to add it back once openbsd-mips64
builds again.
Fixesgolang/go#58110.
Change-Id: Idcd251f0087fa80b5a32a87f18a8b7e9412feda8
Reviewed-on: https://go-review.googlesource.com/c/build/+/463856
Reviewed-by: Bryan Mills <bcmills@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Russ Cox <rsc@golang.org>
Run-TryBot: Russ Cox <rsc@golang.org>
Reviewed-by: Joel Sing <joel@sing.id.au>
Reviewed-by: Carlos Amedee <carlos@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
This flag enables some extra consistency checks that were more
applicable before we added the types2 frontends in Go 1.18. And since
CL 458625, there's nothing left to check anyway so the flag has been
removed.
Fixesgolang/go#58104.
Change-Id: If661d2c5b0033208771ff6b883b72125bfec1dcc
Reviewed-on: https://go-review.googlesource.com/c/build/+/463775
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Bryan C. Mills
Johan Brandhorst-Satzkorn
Dmitri Shuralyov
Jamal Carvalho
Jamal Carvalho
Srinivas Pokala
Michael Anthony Knyszek
cui fliter
Heschi Kreinick
Russ Cox
Paul E. Murphy
Michael Pratt
Tobias Klauser
Jonathan Amsterdam
Eric Grosse
Than McIntosh
Carlos Amedee
Guoqi Chen
Matthew Dempsky