Merge pull request #5923 from AndiDog/empty-err

k8s-ci-robot · web-flow · commit b63abf88fcff · 2026-04-02T02:25:04.000+05:30
🌱 Fix empty error message
diff --git a/pkg/cloud/metrics/metrics.go b/pkg/cloud/metrics/metrics.go
@@ -191,7 +191,6 @@ func getRecordAWSPermissionsIssueMiddleware(target runtime.Object) middleware.Fi
 		if err != nil {
 			request := getContext(ctx)
 			if request != nil {
-				var errMessage string
 				smithyErr := awserrors.ParseSmithyError(err)
 				request.ErrorCode = smithyErr.ErrorCode()
 				switch request.ErrorCode {
@@ -200,7 +199,7 @@ func getRecordAWSPermissionsIssueMiddleware(target runtime.Object) middleware.Fi
 					record.Warnf(target, request.ErrorCode,
 						"Operation %s failed with a credentials or permission issue: %s",
 						request.OperationName,
-						errMessage,
+						smithyErr.ErrorMessage(),
 					)
 				}
 			}

Original file line number	Diff line number	Diff line change
`@@ -191,7 +191,6 @@ func getRecordAWSPermissionsIssueMiddleware(target runtime.Object) middleware.Fi`
`191`	`191`	`if err != nil {`
`192`	`192`	`request := getContext(ctx)`
`193`	`193`	`if request != nil {`
`194`		`- var errMessage string`
`195`	`194`	`smithyErr := awserrors.ParseSmithyError(err)`
`196`	`195`	`request.ErrorCode = smithyErr.ErrorCode()`
`197`	`196`	`switch request.ErrorCode {`
`@@ -200,7 +199,7 @@ func getRecordAWSPermissionsIssueMiddleware(target runtime.Object) middleware.Fi`
`200`	`199`	`record.Warnf(target, request.ErrorCode,`
`201`	`200`	`"Operation %s failed with a credentials or permission issue: %s",`
`202`	`201`	`request.OperationName,`
`203`		`- errMessage,`
	`202`	`+ smithyErr.ErrorMessage(),`
`204`	`203`	`)`
`205`	`204`	`}`
`206`	`205`	`}`