chore: better errors

Signed-off-by: Armando Ruocco <[email protected]>

Author: armru

internal/cmd/healthcheck/main.go

@@ -48,14 +48,21 @@ func unixHealthCheck() *cobra.Command {
 				return healthErr
 			}
 
-			log.Info("received status: %s", res.Status.String())
-			switch res.Status {
-			case grpc_health_v1.HealthCheckResponse_SERVING:
+			if res.Status == grpc_health_v1.HealthCheckResponse_SERVING {
+				log.Trace("healthcheck response OK")
 				os.Exit(0)
+				return nil
+			}
+
+			err := fmt.Errorf("unexpected healthcheck response: %v", res.Status)
+			log.Error(err, "while processing healthcheck response")
+
+			// exit code 1 is returned when we exit from the function with an error
+			switch res.Status {
 			case grpc_health_v1.HealthCheckResponse_UNKNOWN:
-				os.Exit(1)
-			case grpc_health_v1.HealthCheckResponse_NOT_SERVING:
 				os.Exit(2)
+			case grpc_health_v1.HealthCheckResponse_NOT_SERVING:
+				os.Exit(3)
 			default:
 				os.Exit(125)
 			}