dashboard: use n2 instances where necessary

e2 and n2d instances do not support specifying a MinCPUVersion or
NestedVirt.

For golang/go#48969

Change-Id: I0c4b359b29cd24302b869748014f848a9b7b5e39
Reviewed-on: https://go-review.googlesource.com/c/build/+/356071
Trust: Alexander Rakoczy <alex@golang.org>
Run-TryBot: Alexander Rakoczy <alex@golang.org>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
This commit is contained in:
Alexander Rakoczy 2021-10-14 21:54:32 -04:00
Родитель 3025285794
Коммит c6d1678484
1 изменённых файлов: 8 добавлений и 2 удалений

Просмотреть файл

@ -149,6 +149,7 @@ var Hosts = map[string]*HostConfig{
"host-linux-stretch-vmx": &HostConfig{
Notes: "Debian Stretch w/ Nested Virtualization (VMX CPU bit) enabled, for testing",
ContainerImage: "linux-x86-stretch:latest",
machineType: "n2-highcpu-4", // e2 instances do not support MinCPUPlatform or NestedVirt.
NestedVirt: true,
buildletURLTmpl: "http://storage.googleapis.com/$BUCKET/buildlet.linux-amd64",
env: []string{"GOROOT_BOOTSTRAP=/go1.4"},
@ -237,7 +238,7 @@ var Hosts = map[string]*HostConfig{
"host-openbsd-amd64-64": &HostConfig{
VMImage: "openbsd-amd64-64-190129a",
MinCPUPlatform: "Intel Skylake", // for better TSC? Maybe? see Issue 29223. builds faster at least.
machineType: "e2-highcpu-4",
machineType: "n2-highcpu-4", // e2 instances do not support MinCPUPlatform or NestedVirt.
buildletURLTmpl: "https://storage.googleapis.com/$BUCKET/buildlet.openbsd-amd64",
goBootstrapURLTmpl: "https://storage.googleapis.com/$BUCKET/gobootstrap-openbsd-amd64-go1_12.tar.gz",
Notes: "OpenBSD 6.4 with hw.smt=1; GCE VM is built from script in build/env/openbsd-amd64",
@ -721,7 +722,7 @@ type HostConfig struct {
HermeticReverse bool // whether reverse buildlet has fresh env per conn
// Container image options, if ContainerImage != "":
NestedVirt bool // container requires VMX nested virtualization
NestedVirt bool // container requires VMX nested virtualization. e2 and n2d instances are not supported.
KonletVMImage string // optional VM image (containing konlet) to use instead of default
// Optional base env. GOROOT_BOOTSTRAP should go here if the buildlet
@ -1278,6 +1279,11 @@ func (c *HostConfig) MachineType() string {
if v := c.machineType; v != "" {
return v
}
if c.NestedVirt {
// e2 instances do not support nested virtualization, but n2
// instances do.
return "n2-standard-4" // 4 vCPUs, 16 GB mem
}
if c.IsContainer() {
// Set a higher default machine size for containers,
// so their /workdir tmpfs can be larger. The COS