Respect fullnameOverride in Helm chart

Author: Edan-Hamilton

deploy/helm/templates/static-resources.yaml

@@ -3,7 +3,7 @@ kind: ServiceAccount
 metadata:
   labels:
     app: '{{.Release.Name}}'
-  name: security-profiles-operator
+  name: '{{ include "security-profiles-operator.name" . }}'
   namespace: '{{ .Release.Namespace }}'
 ---
 apiVersion: v1
@@ -27,7 +27,7 @@ kind: ClusterRole
 metadata:
   labels:
     app: '{{.Release.Name}}'
-  name: security-profiles-operator
+  name: '{{ include "security-profiles-operator.name" . }}'
 rules:
 - apiGroups:
   - ""
@@ -219,7 +219,7 @@ kind: Role
 metadata:
   labels:
     app: '{{.Release.Name}}'
-  name: security-profiles-operator
+  name: '{{ include "security-profiles-operator.name" . }}'
   namespace: '{{ .Release.Namespace }}'
 rules:
 - apiGroups:
@@ -456,30 +456,30 @@ kind: ClusterRoleBinding
 metadata:
   labels:
     app: '{{.Release.Name}}'
-  name: security-profiles-operator
+  name: '{{ include "security-profiles-operator.name" . }}'
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: ClusterRole
-  name: security-profiles-operator
+  name: '{{ include "security-profiles-operator.name" . }}'
 subjects:
 - kind: ServiceAccount
-  name: security-profiles-operator
+  name: '{{ include "security-profiles-operator.name" . }}'
   namespace: '{{ .Release.Namespace }}'
 ---
 apiVersion: rbac.authorization.k8s.io/v1
 kind: RoleBinding
 metadata:
   labels:
     app: '{{.Release.Name}}'
-  name: security-profiles-operator
+  name: '{{ include "security-profiles-operator.name" . }}'
   namespace: '{{ .Release.Namespace }}'
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: Role
-  name: security-profiles-operator
+  name: '{{ include "security-profiles-operator.name" . }}'
 subjects:
 - kind: ServiceAccount
-  name: security-profiles-operator
+  name: '{{ include "security-profiles-operator.name" . }}'
   namespace: '{{ .Release.Namespace }}'
 ---
 apiVersion: rbac.authorization.k8s.io/v1

deploy/overlays/helm/kustomization.yaml

@@ -39,6 +39,36 @@ patches:
     kind: Deployment
     name: security-profiles-operator-webhook
 
+# Respect fullnameOverride
+- target: { name: security-profiles-operator }
+  patch: |
+    - op: replace
+      path: /metadata/name
+      value: '{{ include "security-profiles-operator.name" . }}'
+- target: { name: security-profiles-operator-webhook }
+  patch: |
+    - op: replace
+      path: /metadata/name
+      value: &name '{{ include "security-profiles-operator.name" . }}-webhook'
+    - op: replace
+      path: /spec/template/metadata/labels/name
+      value: *name
+    - op: replace
+      path: /spec/selector/matchLabels/name
+      value: *name
+- target: { kind: Deployment, name: security-profiles-operator }
+  patch: |
+    - op: replace
+      path: /spec/template/metadata/labels/name
+      value: &name '{{ include "security-profiles-operator.name" . }}'
+    - op: replace
+      path: /spec/selector/matchLabels/name
+      value: *name
+- target: { name: webhook-service }
+  patch: |
+    - op: replace
+      path: /spec/selector/name
+      value: '{{ include "security-profiles-operator.name" . }}-webhook'
 
 # Remove the namespace resource.
 - path: delete-ns.yaml