fix: don't create SA for Admin and View aggregated ClusterRoles (#1551)

Signed-off-by: Jayendra Parsai <[email protected]> Co-authored-by: Jayendra Parsai <[email protected]>
argoproj-labs · Oct 2, 2024 · 8808099 · 8808099
1 parent da6b8a7
commit 8808099
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/controllers/argocd/service_account.go b/controllers/argocd/service_account.go
@@ -78,6 +78,12 @@ func (r *ReconcileArgoCD) reconcileServiceAccounts(cr *argoproj.ArgoCD) error {
 }
 
 func (r *ReconcileArgoCD) reconcileServiceAccountClusterPermissions(name string, rules []v1.PolicyRule, cr *argoproj.ArgoCD) error {
+
+	if name == common.ArgoCDApplicationControllerComponentAdmin || name == common.ArgoCDApplicationControllerComponentView {
+		// Don't create ServiceAccounts
+		return nil
+	}
+
 	var role *v1.ClusterRole
 	var err error