manually pick PrintFailure changes

Signed-off-by: Fabian Gonzalez <fabian.gonzalez@solo.io>

Author: inFocus7

internal/cli/status.go

@@ -35,8 +35,8 @@ func runStatus() error {
 
 	health, err := c.CheckHealth()
 	if err != nil {
-		printer.PrintError(fmt.Sprintf("Daemon is not running (%s)", c.BaseURL))
-		printer.PrintError("Database is not healthy")
+		printer.PrintFailure(fmt.Sprintf("Daemon is not running (%s)", c.BaseURL))
+		printer.PrintFailure("Database is not healthy")
 		return nil
 	}
 
@@ -49,7 +49,7 @@ func runStatus() error {
 	if health.Database == "ok" {
 		printer.PrintSuccess("Database is healthy")
 	} else {
-		printer.PrintError(fmt.Sprintf("Database is not healthy: %s", health.Database))
+		printer.PrintFailure(fmt.Sprintf("Database is not healthy: %s", health.Database))
 	}
 
 	return nil

pkg/printer/printer.go

@@ -46,6 +46,13 @@ func PrintError(message string) {
 	fmt.Fprintf(os.Stderr, "Error: %s\n", message)
 }
 
+// PrintFailure prints a negative-status message with a cross-mark prefix to stdout.
+// Use for reporting expected negative conditions (e.g. "daemon is not running")
+// rather than PrintError which implies an unexpected error.
+func PrintFailure(message string) {
+	_, _ = fmt.Fprintf(os.Stdout, "✗ %s\n", message)
+}
+
 // PrintWarning prints a warning message
 func PrintWarning(message string) {
 	_, _ = fmt.Fprintf(os.Stdout, "Warning: %s\n", message)