feat: add namespace overrides (#471)

Allow to override the default namespace in an Umbrella chart

Co-authored-by: Romeo Dumitrescu <[email protected]>

Author: virtualroot

README.md

@@ -35,6 +35,7 @@ You can customize the values of the helm deployment by using the following Value
 | Parameter                                | Description                                      | Default                                                 |
 | ---------------------------------------- | ------------------------------------------------ | ------------------------------------------------------- |
 | `nameOverride`                           | Overrides release name                           | `""`                                                    |
+| `namespaceOverride`                      | Overrides namespace                              | `""`                                                    |
 | `fullnameOverride`                       | Overrides release fullname                       | `""`                                                    |
 | `image.repository`                       | Container image repository                       | `emberstack/kubernetes-reflector`                       |
 | `image.tag`                              | Container image tag                              | `Same as chart version`                                 |
@@ -165,6 +166,8 @@ spec:
       reflector.v1.k8s.emberstack.com/reflection-allowed-namespaces: ""
   ...
   ```
+
+=======
 > Since version 1.15 of cert-manager you can annotate `Ingress` to create secrets created from certificates for mirroring using `cert-manager.io/secret-template` annotation  (see https://github.com/cert-manager/cert-manager/pull/6839).
 ```yaml
 apiVersion: networking.k8s.io/v1

src/helm/reflector/templates/_helpers.tpl

@@ -30,6 +30,14 @@ Create chart name and version as used by the chart label.
 {{- printf "%s-%s" .Chart.Name .Chart.Version | replace "+" "_" | trunc 63 | trimSuffix "-" }}
 {{- end }}
 
+{{/*
+Expand the namespace of the release.
+Allows overriding it for multi-namespace deployments in combined charts.
+*/}}
+{{- define "reflector.namespace" -}}
+{{- default .Release.Namespace .Values.namespaceOverride | trunc 63 | trimSuffix "-" -}}
+{{- end -}}
+
 {{/*
 Common labels
 */}}

src/helm/reflector/templates/clusterRole.yaml

@@ -3,7 +3,7 @@ apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRole
 metadata:
   name: {{ include "reflector.fullname" . }}
-  namespace: {{ .Release.Namespace }}
+  namespace: {{ include "reflector.namespace" . }}
   labels:
     {{- include "reflector.labels" . | nindent 4 }}
 rules:

src/helm/reflector/templates/clusterRoleBinding.yaml

@@ -3,7 +3,7 @@ apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRoleBinding
 metadata:
   name: {{ include "reflector.fullname" . }}
-  namespace: {{ .Release.Namespace }}
+  namespace: {{ include "reflector.namespace" . }}
   labels:
     {{- include "reflector.labels" . | nindent 4 }}
 roleRef:
@@ -13,5 +13,5 @@ roleRef:
 subjects:
   - kind: ServiceAccount
     name: {{ include "reflector.serviceAccountName" . }}
-    namespace: {{ .Release.Namespace }}
-{{- end }}
+    namespace: {{ include "reflector.namespace" . }}
+{{- end }}

src/helm/reflector/templates/cron.yaml

@@ -4,7 +4,7 @@ apiVersion: {{ .Values.cron.apiVersion | default "batch/v1" }}
 kind: CronJob
 metadata:
   name: {{ include "reflector.fullname" . }}
-  namespace: {{ .Release.Namespace }}
+  namespace: {{ include "reflector.namespace" . }}
   labels:
     {{- include "reflector.labels" . | nindent 4 }}
 

src/helm/reflector/templates/deployment.yaml

@@ -4,7 +4,7 @@ apiVersion: apps/v1
 kind: Deployment
 metadata:
   name: {{ include "reflector.fullname" . }}
-  namespace: {{ .Release.Namespace }}
+  namespace: {{ include "reflector.namespace" . }}
   labels:
     {{- include "reflector.labels" . | nindent 4 }}
 

src/helm/reflector/templates/hpa.yaml

@@ -3,7 +3,7 @@ apiVersion: autoscaling/v2
 kind: HorizontalPodAutoscaler
 metadata:
   name: {{ include "reflector.fullname" . }}
-  namespace: {{ .Release.Namespace }}
+  namespace: {{ include "reflector.namespace" . }}
   labels:
     {{- include "reflector.labels" . | nindent 4 }}
 spec:

src/helm/reflector/templates/serviceaccount.yaml

@@ -3,7 +3,7 @@ apiVersion: v1
 kind: ServiceAccount
 metadata:
   name: {{ include "reflector.serviceAccountName" . }}
-  namespace: {{ .Release.Namespace }}
+  namespace: {{ include "reflector.namespace" . }}
   labels:
     {{- include "reflector.labels" . | nindent 4 }}
   {{- with .Values.serviceAccount.annotations }}

src/helm/reflector/values.yaml

@@ -12,6 +12,7 @@ image:
 
 imagePullSecrets: []
 nameOverride: ""
+namespaceOverride: ""
 fullnameOverride: ""
 
 cron: