Consolidate appends

kubernetes · Mar 31, 2024 · 2a1faaa · 2a1faaa
1 parent 08690f4
commit 2a1faaa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/pkg/resources/aws/aws.go b/pkg/resources/aws/aws.go
@@ -1849,9 +1849,7 @@ func DeleteIAMRole(cloud fi.Cloud, r *resources.Resource) error {
 				}
 				return fmt.Errorf("error listing IAM role policies for %q: %v", roleName, err)
 			}
-			for _, policy := range page.PolicyNames {
-				policyNames = append(policyNames, policy)
-			}
+			policyNames = append(policyNames, page.PolicyNames...)
 		}
 	}
 

diff --git a/upup/pkg/fi/cloudup/awstasks/iamrole.go b/upup/pkg/fi/cloudup/awstasks/iamrole.go
@@ -175,9 +175,7 @@ func (_ *IAMRole) RenderAWS(t *awsup.AWSAPITarget, a, e, changes *IAMRole) error
 					}
 					return fmt.Errorf("error listing IAM role policies: %v", err)
 				}
-				for _, policy := range page.PolicyNames {
-					policyNames = append(policyNames, policy)
-				}
+				policyNames = append(policyNames, page.PolicyNames...)
 			}
 		}