Skip to content

Commit 0be17ea

Browse files
committed
buildlet: correct MinCPUPlatform name
Also, specify a valid CPU Platform for n2 VMs for OpenBSD. See: https://cloud.google.com/compute/docs/instances/specify-min-cpu-platform#availablezones For golang/go#48969 Change-Id: Id503227ad0fbb844e9eaa923b8b3ac7da591ee76 Reviewed-on: https://go-review.googlesource.com/c/build/+/356630 Trust: Alexander Rakoczy <[email protected]> Run-TryBot: Alexander Rakoczy <[email protected]> TryBot-Result: Go Bot <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]>
1 parent 460ff9c commit 0be17ea

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

buildlet/gce.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ func StartNewVM(creds *google.Credentials, buildEnv *buildenv.Environment, instN
7575
minCPU := hconf.MinCPUPlatform
7676
if hconf.IsContainer() {
7777
if hconf.NestedVirt {
78-
minCPU = "Cascade Lake" // n2 vms (which support NestedVirtualization) are either Ice Lake or Cascade Lake.
78+
minCPU = "Intel Cascade Lake" // n2 vms (which support NestedVirtualization) are either Ice Lake or Cascade Lake.
7979
}
8080
if vm := hconf.ContainerVMImage(); vm != "" {
8181
srcImage = "https://www.googleapis.com/compute/v1/projects/" + projectID + "/global/images/" + vm

dashboard/builders.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -237,8 +237,8 @@ var Hosts = map[string]*HostConfig{
237237
},
238238
"host-openbsd-amd64-64": &HostConfig{
239239
VMImage: "openbsd-amd64-64-190129a",
240-
MinCPUPlatform: "Intel Skylake", // for better TSC? Maybe? see Issue 29223. builds faster at least.
241-
machineType: "n2-highcpu-4", // e2 instances do not support MinCPUPlatform or NestedVirt.
240+
MinCPUPlatform: "Intel Cascade Lake", // for better TSC? Maybe? see Issue 29223. builds faster at least.
241+
machineType: "n2-highcpu-4", // e2 instances do not support MinCPUPlatform or NestedVirt.
242242
buildletURLTmpl: "https://storage.googleapis.com/$BUCKET/buildlet.openbsd-amd64",
243243
goBootstrapURLTmpl: "https://storage.googleapis.com/$BUCKET/gobootstrap-openbsd-amd64-go1_12.tar.gz",
244244
Notes: "OpenBSD 6.4 with hw.smt=1; GCE VM is built from script in build/env/openbsd-amd64",

0 commit comments

Comments
 (0)