Merge pull request #2672 from hunterkepley/ocm-12912

OCM-12912 | fix: Combine all policies rather than eating the output
openshift · Dec 3, 2024 · 4cd11d2 · 4cd11d2
2 parents 5feb20c + 04a2765
commit 4cd11d2
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/cmd/dlt/accountroles/cmd.go b/cmd/dlt/accountroles/cmd.go
@@ -237,7 +237,8 @@ func deleteAccountRoles(r *rosa.Runtime, env string, prefix string, clusters []*
 		if roles.CheckIfRolesAreHcpSharedVpc(r, finalRoleList) &&
 			confirm.Prompt(true, "Create commands to delete Hosted CP shared VPC policies?") {
 			for _, role := range finalRoleList {
-				policiesOutput, err = r.AWSClient.GetPolicyDetailsFromRole(awssdk.String(role))
+				policies, err := r.AWSClient.GetPolicyDetailsFromRole(awssdk.String(role))
+				policiesOutput = append(policiesOutput, policies...)
 				if err != nil {
 					r.Reporter.Infof("There was an error getting details of policies attached to role '%s': %v",
 						role, err)