refactor(api/v1beta1/owner_role.go): split cluster role that need to …

…be cluster bound Signed-off-by: Massimiliano Giovagnoli <[email protected]>
projectcapsule · Aug 13, 2022 · 5a9c25b · 5a9c25b
1 parent 3cd7bfe
commit 5a9c25b
Showing 1 changed file with 6 additions and 2 deletions.
diff --git 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 {