commit 4b8b3a91125acde958aa9920df0fecb1dbdb15e9
author Anirudh Mathukumilli <rudymathu@google.com> Tue May 12 23:28:18 2020 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> Tue May 12 23:28:18 2020 +0000
tree dc90e87cc10a77129c797b983d195f20855fecc6
parent d0024341ed4b021c2dcdf93ecce3437e9667b3ad

[health checker] Disable fastboot in health checks

I disabled powercycling into fastboot when fastboot was disabled but
did not fix the corresponding health checks.

Change-Id: Idc76040eae80bdc27994e0bdf531a6dc2ef2a4ee
Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/infra/+/389275
Reviewed-by: Nathan Mulcahey <nmulcahey@google.com>
Commit-Queue: Anirudh Mathukumilli <rudymathu@google.com>

cmd/health_checker/main.go

diff --git a/cmd/health_checker/main.go b/cmd/health_checker/main.go
index 33fd52f..9e92ff2 100644
--- a/cmd/health_checker/main.go
+++ b/cmd/health_checker/main.go
@@ -134,7 +134,7 @@
 func checkHealth(ctx context.Context, n *netboot.Client, device *devicePkg.DeviceTarget) HealthCheckResult {
 	nodename := device.Nodename()
 	log.Printf("Checking health for %s", nodename)
-	if device.Type() == "astro" || device.Type() == "sherlock" {
+	if enableFastboot && (device.Type() == "astro" || device.Type() == "sherlock") {
 		if err := checkIfInFastboot(ctx, device); err != nil {
 			return HealthCheckResult{nodename, unhealthyState, err.Error()}
 		}