From a5c49d8e93f7413be8aef70583abcfa71a561cf6 Mon Sep 17 00:00:00 2001 From: Michel Thebeau Date: Wed, 6 Aug 2025 12:09:11 -0400 Subject: [PATCH] openbao-monitor: remove non-json output of health and init commands Remove the stdout from successful health and init commands. The output of health command will be parsed by scripts. The stdout for init will be extra and not properly formatted for pod logs. Partial-bug: 2117422 Test Plan: PASS inspection of stdout/stderr for health and init commands. Change-Id: I3ff8782399951eaf301fe60372058ade49349514 Signed-off-by: Michel Thebeau --- security/openbao-monitor/source/commands/health.go | 1 - security/openbao-monitor/source/commands/init.go | 1 - 2 files changed, 2 deletions(-) diff --git a/security/openbao-monitor/source/commands/health.go b/security/openbao-monitor/source/commands/health.go index 8912496b..968b5c7e 100644 --- a/security/openbao-monitor/source/commands/health.go +++ b/security/openbao-monitor/source/commands/health.go @@ -44,7 +44,6 @@ var healthCmd = &cobra.Command{ return fmt.Errorf("unable to marshal health check result: %v", err) } slog.Info(fmt.Sprintf("Health check command successful for host %v", args[0])) - fmt.Print("Health check successful. Result:\n") fmt.Print(string(healthPrint)) return nil diff --git a/security/openbao-monitor/source/commands/init.go b/security/openbao-monitor/source/commands/init.go index acfeb739..0df2ee2a 100644 --- a/security/openbao-monitor/source/commands/init.go +++ b/security/openbao-monitor/source/commands/init.go @@ -103,7 +103,6 @@ configurations.`, return fmt.Errorf("Init failed with error: %v", err) } slog.Info(fmt.Sprintf("Init successful for host %v", args[0])) - fmt.Print("Init complete\n") return nil }, PersistentPostRunE: cleanCmd,