From c2c73973044de80b3b775824516e53f0e35623ce Mon Sep 17 00:00:00 2001 From: Andras Jaky Date: Thu, 4 Nov 2021 14:13:04 +0100 Subject: [PATCH] fix: split cluster delete log to different levels - set the log level of the http response to debug - set the log level of the action info message to info - change misleading variable name holding the http response object --- internal/cli/command/cluster/delete.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/internal/cli/command/cluster/delete.go b/internal/cli/command/cluster/delete.go index 3babd67c..a7807190 100644 --- a/internal/cli/command/cluster/delete.go +++ b/internal/cli/command/cluster/delete.go @@ -74,11 +74,12 @@ func runDelete(banzaiCli cli.Cli, options deleteOptions, args []string) error { return errors.New("deletion cancelled") } } - if cluster, err := client.ClustersApi.DeleteCluster(context.Background(), orgId, id, &pipeline.DeleteClusterOpts{Force: optional.NewBool(options.force)}); err != nil { + if resp, err := client.ClustersApi.DeleteCluster(context.Background(), orgId, id, &pipeline.DeleteClusterOpts{Force: optional.NewBool(options.force)}); err != nil { cli.LogAPIError("delete cluster", err, id) return errors.WrapIf(err, "failed to delete cluster") } else { - log.Printf("Deleting cluster %v", cluster) + log.Info("deleting cluster") + log.Debug(resp) } if cluster, _, err := client.ClustersApi.GetCluster(context.Background(), orgId, id); err != nil { cli.LogAPIError("get cluster", err, id)