From 5a9c25b125a49e9824f0899e1be762a0e0da77fe Mon Sep 17 00:00:00 2001
From: Massimiliano Giovagnoli <me@maxgio.it>
Date: Sat, 13 Aug 2022 18:56:35 +0200
Subject: [PATCH] refactor(api/v1beta1/owner_role.go): split cluster role that
 need to be cluster bound

Signed-off-by: Massimiliano Giovagnoli <me@maxgio.it>
---
 api/v1beta1/owner_role.go | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/api/v1beta1/owner_role.go b/api/v1beta1/owner_role.go
index 74d3697a..da9e6cb0 100644
--- a/api/v1beta1/owner_role.go
+++ b/api/v1beta1/owner_role.go
@@ -40,11 +40,15 @@ func (in OwnerSpec) GetRoles(tenant Tenant, index int) []string {
 
 	roles := []string{"admin", "capsule-namespace-deleter"}
 
+	return roles
+}
+
+func (in OwnerSpec) GetClusterRoles(tenant Tenant) []string {
 	if tenant.Spec.GitOpsReady {
-		roles = append(roles, in.getGitOpsRoles(tenant)...)
+		return in.getGitOpsRoles(tenant)
 	}
 
-	return roles
+	return []string{}
 }
 
 func (in OwnerSpec) getGitOpsRoles(tenant Tenant) []string {