Skip to content

Commit 2a46f91

Browse files
committed
Fix uninstall hook
1 parent 607b6a4 commit 2a46f91

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

helm/designate-certmanager-webhook/templates/uninstall.yaml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,15 +7,15 @@ metadata:
77
chart: {{ include "designate-certmanager-webhook.chart" . }}
88
release: {{ .Release.Name }}
99
heritage: {{ .Release.Service }}
10-
namespace: default
10+
namespace: {{ .Release.Namespace }}
1111
annotations:
1212
"helm.sh/hook": pre-delete
1313
"helm.sh/hook-delete-policy": hook-succeeded
1414
spec:
1515
template:
1616
spec:
1717
restartPolicy: Never
18-
serviceAccountName: {{ include "designate-certmanager-webhook.fullname" . }}
18+
serviceAccountName: {{ include "designate-certmanager-webhook.fullname" . }}-uninstall
1919
containers:
2020
- name: remove-apiservice
2121
image: "{{ .Values.kubectl.image.repository }}:{{ .Values.kubectl.image.tag }}"
@@ -74,13 +74,13 @@ roleRef:
7474
subjects:
7575
- apiGroup: ""
7676
kind: ServiceAccount
77-
name: {{ include "designate-certmanager-webhook.fullname" . }}
78-
namespace: default
77+
name: {{ include "designate-certmanager-webhook.fullname" . }}-uninstall
78+
namespace: {{ .Release.Namespace }}
7979
---
8080
apiVersion: v1
8181
kind: ServiceAccount
8282
metadata:
83-
name: {{ include "designate-certmanager-webhook.fullname" . }}
83+
name: {{ include "designate-certmanager-webhook.fullname" . }}-uninstall
8484
labels:
8585
app: {{ include "designate-certmanager-webhook.name" . }}
8686
chart: {{ include "designate-certmanager-webhook.chart" . }}
@@ -89,4 +89,4 @@ metadata:
8989
annotations:
9090
"helm.sh/hook": pre-delete
9191
"helm.sh/hook-delete-policy": hook-succeeded
92-
namespace: default
92+
namespace: {{ .Release.Namespace }}

0 commit comments

Comments
 (0)