Skip to content

Commit d8ae6e4

Browse files
committed
set database namespace to match watch namespace by default
1 parent 21f7614 commit d8ae6e4

File tree

3 files changed

+30
-14
lines changed

3 files changed

+30
-14
lines changed

charts/community-operator/templates/_helpers.tpl

+13
Original file line numberDiff line numberDiff line change
@@ -10,3 +10,16 @@ Operator's watch namespaces
1010
{{- $namespaces | toYaml }}
1111
{{- end }}
1212
{{- end }}
13+
14+
{{/*
15+
Database namespaces
16+
*/}}
17+
{{- define "community-operator.database.namespaces" -}}
18+
{{- $defaultNamespaces := include "community-operator.watchNamespaces" . | fromYamlArray }}
19+
{{- $namespaces := default $defaultNamespaces .Values.database.namespaces }}
20+
{{- if has "*" $namespaces }}
21+
{{- list | toYaml }}
22+
{{- else }}
23+
{{- $namespaces | toYaml }}
24+
{{- end }}
25+
{{- end }}

charts/community-operator/templates/database_roles.yaml

+14-8
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,20 @@
1+
{{- $databaseNamespaces := include "community-operator.database.namespaces" . | fromYamlArray }}
2+
{{- $databaseName := $.Values.database.name }}
3+
{{- range $namespace := $databaseNamespaces }}
4+
15
---
26
apiVersion: v1
37
kind: ServiceAccount
48
metadata:
5-
name: {{ .Values.database.name }}
6-
namespace: {{ if .Values.database.namespace }} {{ .Values.database.namespace }} {{ else }} {{ .Release.Namespace }} {{ end }}
9+
name: {{ $databaseName }}
10+
namespace: {{ $namespace }}
711

812
---
913
apiVersion: rbac.authorization.k8s.io/v1
1014
kind: Role
1115
metadata:
12-
name: {{ .Values.database.name }}
13-
namespace: {{ if .Values.database.namespace }} {{ .Values.database.namespace }} {{ else }} {{ .Release.Namespace }} {{ end }}
16+
name: {{ $databaseName }}
17+
namespace: {{ $namespace }}
1418
rules:
1519
- apiGroups:
1620
- ""
@@ -31,12 +35,14 @@ rules:
3135
kind: RoleBinding
3236
apiVersion: rbac.authorization.k8s.io/v1
3337
metadata:
34-
name: {{ .Values.database.name }}
35-
namespace: {{ if .Values.database.namespace }} {{ .Values.database.namespace }} {{ else }} {{ .Release.Namespace }} {{ end }}
38+
name: {{ $databaseName }}
39+
namespace: {{ $namespace }}
3640
subjects:
3741
- kind: ServiceAccount
38-
name: {{ .Values.database.name }}
42+
name: {{ $databaseName }}
3943
roleRef:
4044
kind: Role
41-
name: {{ .Values.database.name }}
45+
name: {{ $databaseName }}
4246
apiGroup: rbac.authorization.k8s.io
47+
48+
{{- end }}

charts/community-operator/values.yaml

+3-6
Original file line numberDiff line numberDiff line change
@@ -53,14 +53,11 @@ operator:
5353
## Operator's database
5454
database:
5555
name: mongodb-database
56-
# set this to the namespace where you would like
57-
# to deploy the MongoDB database,
58-
# Note if the database namespace is not same
59-
# as the operator namespace,
60-
# make sure to set "watchNamespace" to "*"
56+
# If "watchNamespaces" is set to ["*"],
57+
# then set the database namespaces
6158
# to ensure that the operator has the
6259
# permission to reconcile resources in other namespaces
63-
# namespace: mongodb-database
60+
# namespaces: ["mongodb-database"]
6461

6562
agent:
6663
name: mongodb-agent-ubi

0 commit comments

Comments
 (0)