Skip to content

Commit

Permalink
all: remove packet arm64 machines
Browse files Browse the repository at this point in the history
linux-arm64-packet builder machine is getting decommissioned on
27/06/22. This CL removes the build machine before the decommission.

Fixes golang/go#53436

Change-Id: I9992481bddc3376d86ec0d69d8e7da0157333adf
Reviewed-on: https://go-review.googlesource.com/c/build/+/413154
Run-TryBot: Carlos Amedee <carlos@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Carlos Amedee <carlos@golang.org>
  • Loading branch information
aaqaishtyaq authored and gopherbot committed Jun 21, 2022
1 parent 4de31d1 commit cef0fbf
Show file tree
Hide file tree
Showing 7 changed files with 1 addition and 123 deletions.
12 changes: 1 addition & 11 deletions cmd/buildlet/stage0/stage0.go
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ func main() {
}
case "linux/arm64":
switch env := os.Getenv("GO_BUILDER_ENV"); env {
case "host-linux-arm64-packet", "host-linux-arm64-aws":
case "host-linux-arm64-aws":
// No special setup.
default:
panic(fmt.Sprintf("unknown/unspecified $GO_BUILDER_ENV value %q", env))
Expand Down Expand Up @@ -182,16 +182,6 @@ Download:
cmd.Args = append(cmd.Args, reverseHostTypeArgs("host-linux-s390x")...)
case "linux/arm64":
switch buildEnv {
case "host-linux-arm64-packet":
hostname := os.Getenv("HOSTNAME") // if empty, docker container name is used
cmd.Args = append(cmd.Args,
"--reverse-type="+buildEnv,
"--workdir=/workdir",
"--hostname="+hostname,
"--halt=false",
"--reboot=false",
"--coordinator=farmer.golang.org:443",
)
case "host-linux-arm64-aws":
// no special configuration
default:
Expand Down
15 changes: 0 additions & 15 deletions cmd/coordinator/status.go
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,6 @@ var basePinErr atomic.Value
func addHealthCheckers(ctx context.Context, sc *secret.Client) {
addHealthChecker(newMacHealthChecker())
addHealthChecker(newMacOSARM64Checker())
addHealthChecker(newPacketHealthChecker())
addHealthChecker(newOSUPPC64Checker())
addHealthChecker(newOSUPPC64leChecker())
addHealthChecker(newOSUPPC64lePower9Checker())
Expand Down Expand Up @@ -399,20 +398,6 @@ func expectedHosts(hostType string) int {
return hc.ExpectNum
}

func newPacketHealthChecker() *healthChecker {
var hosts []string
for i := 1; i <= expectedHosts("host-linux-arm64-packet"); i++ {
name := fmt.Sprintf("packet%02d", i)
hosts = append(hosts, name)
}
return &healthChecker{
ID: "packet",
Title: "Packet linux/arm64 machines",
DocURL: "https://github.com/golang/build/tree/master/env/linux-arm64/packet",
Check: reverseHostChecker(hosts),
}
}

func newOSUPPC64Checker() *healthChecker {
var hosts []string
for i := 1; i <= expectedHosts("host-linux-ppc64-osu"); i++ {
Expand Down
13 changes: 0 additions & 13 deletions dashboard/builders.go
Original file line number Diff line number Diff line change
Expand Up @@ -564,14 +564,6 @@ var Hosts = map[string]*HostConfig{
SSHUsername: "root",
HermeticReverse: true,
},
"host-linux-arm64-packet": &HostConfig{
Notes: "On 96 core packet.net host (Xenial) in Docker containers (Debian Buster); run by Go team. See x/build/env/linux-arm64/packet",
IsReverse: true,
HermeticReverse: true,
ExpectNum: 8,
env: []string{"GOROOT_BOOTSTRAP=/usr/local/go-bootstrap"},
SSHUsername: "root",
},
"host-linux-arm64-aws": &HostConfig{
Notes: "Debian Buster, EC2 arm64 instance. See x/build/env/linux-arm64/aws",
VMImage: "ami-03089323a1d38e652",
Expand Down Expand Up @@ -2528,11 +2520,6 @@ func init() {
distTestAdjust: ppc64DistTestPolicy,
env: []string{"GO_TEST_TIMEOUT_SCALE=2"}, // see golang.org/issues/44422
})
addBuilder(BuildConfig{
Name: "linux-arm64-packet",
HostType: "host-linux-arm64-packet",
FlakyNet: true, // maybe not flaky, but here conservatively
})
addBuilder(BuildConfig{
Name: "linux-arm64-aws",
HostType: "host-linux-arm64-aws",
Expand Down
9 changes: 0 additions & 9 deletions env/linux-arm64/packet/Dockerfile

This file was deleted.

52 changes: 0 additions & 52 deletions env/linux-arm64/packet/README

This file was deleted.

10 changes: 0 additions & 10 deletions env/linux-arm64/packet/build.sh

This file was deleted.

13 changes: 0 additions & 13 deletions env/linux-arm64/packet/rundockerbuildlet.service

This file was deleted.

0 comments on commit cef0fbf

Please sign in to comment.