Skip to content

Commit

Permalink
dashboard: restrict CPU usage on packet builders
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
toothrot committed Dec 19, 2019
1 parent 8a72940 commit f9092bb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion dashboard/builders.go
Original file line number Diff line number Diff line change
Expand Up @@ -564,7 +564,7 @@ var Hosts = map[string]*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: 20,
ExpectNum: 8,
env: []string{"GOROOT_BOOTSTRAP=/usr/local/go-bootstrap"},
SSHUsername: "root",
},
Expand Down
2 changes: 1 addition & 1 deletion env/linux-arm64/packet/rundockerbuildlet.service
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ WantedBy=network-online.target

[Service]
Type=simple
ExecStart=/usr/local/bin/rundockerbuildlet -basename=packet -image=gobuilder-arm64-packet:1 -n=20
ExecStart=/usr/local/bin/rundockerbuildlet -basename=packet -image=gobuilder-arm64-packet:1 -n=8 -cpu=12
Restart=always
RestartSec=2
StartLimitInterval=0

0 comments on commit f9092bb

Please sign in to comment.