JustScreaMy/cert-manager-webhook-porkbun
1
0
Fork 0
forked from KropCloud/cert-manager-webhook-example
Code Pull requests Activity

Add explicit namespaces to all namespaced resources in Helm charts

Browse source 
A few resources didn't specify a namespace, resulting in a discrepency if `helm template --namespace` is ran and later `kubectl apply` is in a different context.

Signed-off-by: James Lakin <james@jameslakin.co.uk>
This commit is contained in:
James Lakin 2022-04-16 10:19:05 +01:00
parent be0eb944c4
commit 8909f58328
3 changed files with 3 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
deploy/example-webhook/templates/deployment.yaml
View file

@ -2,6 +2,7 @@ apiVersion: apps/v1
kind: Deployment
metadata:
name: {{ include "example-webhook.fullname" . }}
namespace: {{ .Release.Namespace | quote }}
labels:
app: {{ include "example-webhook.name" . }}
chart: {{ include "example-webhook.chart" . }}

1
deploy/example-webhook/templates/rbac.yaml
View file

@ -2,6 +2,7 @@ apiVersion: v1
kind: ServiceAccount
metadata:
name: {{ include "example-webhook.fullname" . }}
namespace: {{ .Release.Namespace | quote }}
labels:
app: {{ include "example-webhook.name" . }}
chart: {{ include "example-webhook.chart" . }}

1
deploy/example-webhook/templates/service.yaml
View file

@ -2,6 +2,7 @@ apiVersion: v1
kind: Service
metadata:
name: {{ include "example-webhook.fullname" . }}
namespace: {{ .Release.Namespace | quote }}
labels:
app: {{ include "example-webhook.name" . }}
chart: {{ include "example-webhook.chart" . }}