Skip to content

Commit

Permalink
Remove dependency to clusterType
Browse files Browse the repository at this point in the history
  • Loading branch information
lvarin committed Jan 30, 2024
1 parent f22c09a commit 07fcc09
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 20 deletions.
18 changes: 1 addition & 17 deletions charts/tesk/templates/common/taskmaster-rbac.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,20 +3,6 @@ apiVersion: v1
kind: ServiceAccount
metadata:
name: taskmaster
{{ if eq .Values.clusterType "openshift" }}
---
kind: RoleBinding
apiVersion: rbac.authorization.k8s.io/v1
metadata:
name: taskmaster-role-binding
subjects:
- kind: ServiceAccount
name: taskmaster
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: ClusterRole
name: admin
{{ else }}
---
apiVersion: rbac.authorization.k8s.io/v1
kind: Role
Expand All @@ -34,6 +20,7 @@ rules:
- ""
resources:
- persistentvolumeclaims
- configmaps
verbs:
- create
- delete
Expand All @@ -46,7 +33,6 @@ rules:
verbs:
- get
- list
- delete
- apiGroups:
- batch
resources:
Expand All @@ -69,5 +55,3 @@ roleRef:
kind: Role
name: taskmaster
apiGroup: rbac.authorization.k8s.io
{{ end }}

2 changes: 1 addition & 1 deletion charts/tesk/templates/openshift/oc-route.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
{{ if eq .Values.clusterType "openshift" }}
{{ if .Capabilities.APIVersions.Has "route.openshift.io/v1" }}
apiVersion: route.openshift.io/v1
kind: Route
metadata:
Expand Down
2 changes: 0 additions & 2 deletions charts/tesk/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@

# host_name: tes.tsi.ebi.ac.uk
host_name: ""
# There are two options here: "openshift" or "kubernetes"
clusterType: kubernetes
#
#
#
Expand Down

0 comments on commit 07fcc09

Please sign in to comment.