Skip to content

Commit 157d745

Browse files
committed
Added namespaceOverride template variable to support flexible namespace configurations
Signed-off-by: Florian Geckeler <[email protected]>
1 parent f256955 commit 157d745

File tree

9 files changed

+34
-7
lines changed

9 files changed

+34
-7
lines changed

Diff for: charts/kube-vip-cloud-provider/templates/_helpers.tpl

+11
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,17 @@ Expand the name of the chart.
66
{{- default .Chart.Name .Values.nameOverride | trunc 63 | trimSuffix "-" }}
77
{{- end }}
88

9+
{{/*
10+
Determine the namespace to use, allowing for a namespace override.
11+
*/}}
12+
{{- define "kube-vip-cloudprovider.namespace" -}}
13+
{{- if .Values.namespaceOverride }}
14+
{{- .Values.namespaceOverride }}
15+
{{- else }}
16+
{{- .Release.Namespace }}
17+
{{- end }}
18+
{{- end }}
19+
920
{{/*
1021
Create a default fully qualified app name.
1122
We truncate at 63 chars because some Kubernetes name fields are limited to this (by the DNS naming spec).

Diff for: charts/kube-vip-cloud-provider/templates/configmap.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ apiVersion: v1
33
kind: ConfigMap
44
metadata:
55
name: {{ include "kube-vip-cloud-provider.name" . }}
6-
namespace: {{ .Release.Namespace }}
6+
namespace: {{ include "kube-vip-cloudprovider.namespace" . }}
77
data:
88
{{- range $key, $value := .Values.cm.data }}
99
{{ $key }}: {{ $value | quote }}

Diff for: charts/kube-vip-cloud-provider/templates/deployment.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ apiVersion: apps/v1
22
kind: Deployment
33
metadata:
44
name: {{ include "kube-vip-cloud-provider.name" . }}
5-
namespace: {{ .Release.Namespace | default "kube-system" }}
5+
namespace: {{ include "kube-vip-cloudprovider.namespace" . | default "kube-system" }}
66
spec:
77
replicas: {{ .Values.replicasCount }}
88
selector:

Diff for: charts/kube-vip-cloud-provider/templates/rbac.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ apiVersion: v1
22
kind: ServiceAccount
33
metadata:
44
name: {{ include "kube-vip-cloud-provider.name" . }}
5-
namespace: {{ .Release.Namespace | default "kube-system" }}
5+
namespace: {{ include "kube-vip-cloudprovider.namespace" . | default "kube-system" }}
66
labels:
77
{{- include "kube-vip-cloud-provider.labels" . | nindent 4 }}
88
---
@@ -34,4 +34,4 @@ roleRef:
3434
subjects:
3535
- kind: ServiceAccount
3636
name: {{ include "kube-vip-cloud-provider.name" . }}
37-
namespace: {{ .Release.Namespace }}
37+
namespace: {{ include "kube-vip-cloudprovider.namespace" . }}

Diff for: charts/kube-vip-cloud-provider/values.yaml

+3
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@ image:
1010
# Overrides the image tag whose default is the chart appVersion.
1111
# tag: "v0.0.4"
1212

13+
# Custom namespace to override the namespace for the deployed resources.
14+
namespaceOverride: ""
15+
1316
## Cloud Provider ConfigMap
1417
## CIDR blocks , IP ranges [start address - end address]
1518
## Multiple pools by CIDR per Namespace, Multiple IP ranges per Namespace (handles overlapping ranges)

Diff for: charts/kube-vip/templates/_helpers.tpl

+11
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,17 @@ If release name contains chart name it will be used as a full name.
2424
{{- end }}
2525
{{- end }}
2626

27+
{{/*
28+
Determine the namespace to use, allowing for a namespace override.
29+
*/}}
30+
{{- define "kube-vip.namespace" -}}
31+
{{- if .Values.namespaceOverride }}
32+
{{- .Values.namespaceOverride }}
33+
{{- else }}
34+
{{- .Release.Namespace }}
35+
{{- end }}
36+
{{- end }}
37+
2738
{{/*
2839
Create chart name and version as used by the chart label.
2940
*/}}

Diff for: charts/kube-vip/templates/daemonset.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ apiVersion: apps/v1
22
kind: DaemonSet
33
metadata:
44
name: {{ include "kube-vip.name" . }}
5-
namespace: {{ .Release.Namespace | default "kube-system" }}
5+
namespace: {{ include "kube-vip.namespace" . | default "kube-system" }}
66
spec:
77
selector:
88
matchLabels:

Diff for: charts/kube-vip/templates/rbac.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ apiVersion: v1
22
kind: ServiceAccount
33
metadata:
44
name: {{ include "kube-vip.name" . }}
5-
namespace: {{ .Release.Namespace }}
5+
namespace: {{ include "kube-vip.namespace" . }}
66
labels:
77
{{- include "kube-vip.labels" . | nindent 4 }}
88
---
@@ -33,4 +33,4 @@ roleRef:
3333
subjects:
3434
- kind: ServiceAccount
3535
name: {{ include "kube-vip.name" . }}
36-
namespace: {{ .Release.Namespace }}
36+
namespace: {{ include "kube-vip.namespace" . }}

Diff for: charts/kube-vip/values.yaml

+2
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,8 @@ extraLabels: {}
5555
imagePullSecrets: []
5656
nameOverride: ""
5757
fullnameOverride: ""
58+
# Custom namespace to override the namespace for the deployed resources.
59+
namespaceOverride: ""
5860

5961
serviceAccount:
6062
# Specifies whether a service account should be created

0 commit comments

Comments
 (0)