Update with checks.

konstructio · Jan 16, 2025 · 4f2b84e · 4f2b84e
1 parent 6d83d83
commit 4f2b84e
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/cmd/aws/create.go b/cmd/aws/create.go
@@ -304,7 +304,8 @@ func createKubernetesAdminRole(ctx context.Context, clusterName string, iamClien
 	// Check if the IAM policy exists
 	cp, err := iamClient.GetPolicy(ctx, &iam.GetPolicyInput{PolicyArn: aws.String(fmt.Sprintf("arn:aws:iam::%s:policy/%s", *callerIdentity.Account, policyName))})
 	if err != nil {
-		if newError := (&awshttp.ResponseError{}); errors.As(err, newError) && newError.HTTPStatusCode() == http.StatusNotFound {
+		var newError awshttp.ResponseError
+		if errors.As(err, &newError) && newError.HTTPStatusCode() == http.StatusNotFound {
 			// Policy does not exist, continue
 		} else {
 			return "", fmt.Errorf("failed to get policy %q: %w", policyName, err)
@@ -352,7 +353,8 @@ func createKubernetesAdminRole(ctx context.Context, clusterName string, iamClien
 	// Check if a role with this name already exists
 	role, err := iamClient.GetRole(ctx, &iam.GetRoleInput{RoleName: aws.String(roleName)})
 	if err != nil {
-		if newError := (&awshttp.ResponseError{}); errors.As(err, newError) && newError.HTTPStatusCode() == http.StatusNotFound {
+		var newError awshttp.ResponseError
+		if errors.As(err, &newError) && newError.HTTPStatusCode() == http.StatusNotFound {
 			// Role does not exist, continue
 		} else {
 			return "", fmt.Errorf("failed to get role %q: %w", roleName, err)