Skip to content

Commit 0f149ef

Browse files
committed
Add a way to override Namespace for metabase chart
1 parent bbf7559 commit 0f149ef

14 files changed

Lines changed: 22 additions & 16 deletions

charts/metabase/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ description:
33
The easy, open source way for everyone in your company to ask questions
44
and learn from data.
55
name: metabase
6-
version: 2.24.2
6+
version: 2.24.3
77
appVersion: v0.56.20.x
88
maintainers:
99
- name: pmint93

charts/metabase/templates/_helpers.tpl

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,12 @@ Create chart name and version as used by the chart label.
8686

8787
{{/*
8888
Create the namespace name
89+
We truncate at 63 chars because some Kubernetes name fields are limited to this (by the DNS naming spec).
8990
*/}}
9091
{{- define "metabase.namespace" -}}
91-
{{- .Release.Namespace -}}
92+
{{- if .Values.fullnamespaceOverride -}}
93+
{{- .Values.fullnamespaceOverride | trunc 63 | trimSuffix "-" -}}
94+
{{- else -}}
95+
{{- printf .Release.Namespace | trunc 63 | trimSuffix "-" -}}
96+
{{- end -}}
9297
{{- end -}}

charts/metabase/templates/config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ apiVersion: v1
22
kind: ConfigMap
33
metadata:
44
name: {{ template "metabase.fullname" . }}-config
5-
namespace: {{ .Release.Namespace }}
5+
namespace: {{ template "metabase.namespace" . }}
66
labels:
77
app: {{ template "metabase.name" . }}
88
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}

charts/metabase/templates/database-secret.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ apiVersion: v1
33
kind: Secret
44
metadata:
55
name: {{ template "metabase.fullname" . }}-database
6-
namespace: {{ .Release.Namespace }}
6+
namespace: {{ template "metabase.namespace" . }}
77
labels:
88
app: {{ template "metabase.name" . }}
99
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}

charts/metabase/templates/deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ apiVersion: {{ template "deployment.apiVersion" . }}
22
kind: Deployment
33
metadata:
44
name: {{ template "metabase.fullname" . }}
5-
namespace: {{ .Release.Namespace }}
5+
namespace: {{ template "metabase.namespace" . }}
66
labels:
77
app: {{ template "metabase.name" . }}
88
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}

charts/metabase/templates/ingress.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ apiVersion: networking.k8s.io/v1
88
kind: Ingress
99
metadata:
1010
name: {{ template "metabase.fullname" . }}
11-
namespace: {{ .Release.Namespace }}
11+
namespace: {{ template "metabase.namespace" . }}
1212
labels:
1313
app: {{ template "metabase.name" . }}
1414
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}
@@ -47,7 +47,7 @@ apiVersion: extensions/v1beta1
4747
kind: Ingress
4848
metadata:
4949
name: {{ template "metabase.fullname" . }}
50-
namespace: {{ .Release.Namespace }}
50+
namespace: {{ template "metabase.namespace" . }}
5151
labels:
5252
app: {{ template "metabase.name" . }}
5353
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}

charts/metabase/templates/pdb.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ apiVersion: policy/v1
33
kind: PodDisruptionBudget
44
metadata:
55
name: {{ template "metabase.fullname" . }}
6-
namespace: {{ .Release.Namespace }}
6+
namespace: {{ template "metabase.namespace" . }}
77
labels:
88
app: {{ template "metabase.name" . }}
99
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}

charts/metabase/templates/route.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ apiVersion: route.openshift.io/v1
44
kind: Route
55
metadata:
66
name: {{ template "metabase.fullname" . }}
7-
namespace: {{ .Release.Namespace }}
7+
namespace: {{ template "metabase.namespace" . }}
88
labels:
99
app: {{ template "metabase.name" . }}
1010
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}

charts/metabase/templates/securitygrouppolicy.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ apiVersion: vpcresources.k8s.aws/v1beta1
33
kind: SecurityGroupPolicy
44
metadata:
55
name: {{ template "metabase.fullname" . }}
6-
namespace: {{ .Release.Namespace }}
6+
namespace: {{ template "metabase.namespace" . }}
77
labels:
88
app: {{ template "metabase.name" . }}
99
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}

charts/metabase/templates/service.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ apiVersion: v1
22
kind: Service
33
metadata:
44
name: {{ template "metabase.fullname" . }}
5-
namespace: {{ .Release.Namespace }}
5+
namespace: {{ template "metabase.namespace" . }}
66
labels:
77
app: {{ template "metabase.name" . }}
88
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}
@@ -47,4 +47,4 @@ spec:
4747
{{- end }}
4848
selector:
4949
app: {{ template "metabase.name" . }}
50-
release: {{ .Release.Name }}
50+
release: {{ .Release.Name }}

0 commit comments

Comments
 (0)