[health checker] Powercycle don't change health
Powercycles aren't guaranteed to succeed, so devices should not
be marked as healthy.
Change-Id: I20d074479e9c9db62d54d90bbc965ea52eac336e
diff --git a/cmd/health_checker/main.go b/cmd/health_checker/main.go
index 72143ce..927c322 100644
--- a/cmd/health_checker/main.go
+++ b/cmd/health_checker/main.go
@@ -152,12 +152,10 @@
log.Printf("state=%s, error_msg=%s", checkResult.State, checkResult.ErrorMsg)
if checkResult.State == unhealthyState && rebootIfUnhealthy {
if err := device.Powercycle(ctx); err != nil {
- log.Printf("reboot failed with error: %s", err.Error())
+ log.Printf("powercycle call failed with error: %s", err.Error())
checkResult.ErrorMsg += "; " + err.Error()
} else {
- log.Printf("reboot succeeded for %s", device.Nodename())
- checkResult.State = "healthy"
- checkResult.ErrorMsg = ""
+ log.Printf("powercycle call succeeded for %s", device.Nodename())
}
}
checkResultSlice = append(checkResultSlice, checkResult)