fix: missing stdout on dry run/apply

replicatedhq · Jul 23, 2021 · 9d77da9 · 9d77da9
1 parent aec092a
commit 9d77da9
Showing 1 changed file with 1 addition and 14 deletions.
diff --git a/kotsadm/operator/pkg/client/client.go b/kotsadm/operator/pkg/client/client.go
@@ -235,20 +235,7 @@ func (c *Client) registerHandlers(socketClient *socket.Client) error {
 		var helmResult *commandResult
 		var deployError, helmError error
 		defer func() {
-			var dryRunResult, applyResult *commandResult
-
-			// only send results if we have an error
-			if deployRes.dryRunResult.hasErr {
-				dryRunResult = &deployRes.dryRunResult
-			}
-			if deployRes.applyResult.hasErr {
-				applyResult = &deployRes.applyResult
-			}
-			if helmResult != nil && !helmResult.hasErr {
-				helmResult = &commandResult{}
-			}
-
-			err := c.sendResult(args, dryRunResult, applyResult, helmResult)
+			err := c.sendResult(args, &deployRes.dryRunResult, &deployRes.applyResult, helmResult)
 			if err != nil {
 				log.Printf("failed to report result: %v", err)
 			}