Skip to content

Commit

Permalink
fix: don't create SA for Admin and View aggregated ClusterRoles
Browse files Browse the repository at this point in the history
Signed-off-by: Jayendra Parsai <[email protected]>
  • Loading branch information
Jayendra Parsai committed Oct 1, 2024
1 parent da6b8a7 commit b5c5f8c
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions controllers/argocd/service_account.go
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down

0 comments on commit b5c5f8c

Please sign in to comment.