Merge pull request #7115 from a-hilaly/logs-wv

Refine log statements using `%w` for better formatting and UX

integration/tests/params.go

@@ -272,7 +272,7 @@ func (p *Params) attemptSettingUserID() {
 		},
 	})
 
-	msg := "Warning: failed to get UserID, proceeding without configuring cluster name prefix. Error: %w"
+	msg := "Warning: failed to get UserID, proceeding without configuring cluster name prefix. Error: %v"
 	if err != nil {
 		fmt.Fprintf(GinkgoWriter, msg, err)
 		return

pkg/actions/cluster/delete.go

@@ -219,6 +219,6 @@ func attemptVpcCniDeletion(ctx context.Context, clusterConfig *api.ClusterConfig
 	logger.Debug("deleting kube-system/aws-node DaemonSet")
 	err := clientSet.AppsV1().DaemonSets("kube-system").Delete(ctx, "aws-node", metav1.DeleteOptions{})
 	if err != nil {
-		logger.Debug("failed to delete kube-system/aws-node DaemonSet: %w", err)
+		logger.Debug("failed to delete kube-system/aws-node DaemonSet: %v", err)
 	}
 }

pkg/actions/nodegroup/drain.go

@@ -42,7 +42,7 @@ func (m *Manager) Drain(ctx context.Context, input *DrainInput) error {
 	}
 	err := g.Wait()
 	if err != nil {
-		logger.Critical("Node group drain failed: %w", err)
+		logger.Critical("Node group drain failed: %v", err)
 	}
 	waitForAllRoutinesToFinish(ctx, sem, parallelLimit)
 	return err

pkg/actions/nodegroup/get.go

@@ -250,7 +250,7 @@ func (m *Manager) mapStackToNodeGroupSummary(ctx context.Context, stack *manager
 		}
 		collectorSet := outputs.NewCollectorSet(collectors)
 		if err := collectorSet.MustCollect(*stack); err != nil {
-			logger.Warning(fmt.Errorf("error collecting Cloudformation outputs for stack %s: %w", *stack.StackName, err).Error())
+			logger.Warning("error collecting Cloudformation outputs for stack %s: %v", *stack.StackName, err)
 		}
 	}