Merge pull request #96 from draychev/simplify-3

Simplify the usage of ParseIRSAAnnotation
keikoproj · May 7, 2022 · a7120cf · a7120cf
2 parents f72bd8e + 998e835
commit a7120cf
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/internal/utils/utils.go b/internal/utils/utils.go
@@ -22,11 +22,8 @@ func GetTrustPolicy(ctx context.Context, role *iammanagerv1alpha1.Iamrole) (stri
 	var statements []iammanagerv1alpha1.TrustPolicyStatement
 
 	// Is it IRSA use case
-	flag, saName := ParseIRSAAnnotation(ctx, role)
-
-	//Construct AssumeRoleWithWebIdentity
-	if flag {
-
+	// Construct AssumeRoleWithWebIdentity
+	if flag, saName := ParseIRSAAnnotation(ctx, role); flag {
 		hostPath := fmt.Sprintf("%s", strings.TrimPrefix(config.Props.OIDCIssuerUrl(), "https://"))
 		statement := iammanagerv1alpha1.TrustPolicyStatement{
 			Effect: "Allow",