Added liveness and readiness probes to helm chart. Removed resource limits from olm template.

Author: trevorbox

README.md

@@ -263,13 +263,13 @@ Deploy chart...
 
 ```shell
 make helmchart IMG=${imageRepository} VERSION=${imageTag}
-helm upgrade -i namespace-configuration-operator-helmchart-test charts/namespace-configuration-operator -n namespace-configuration-operator-local --create-namespace
+helm upgrade -i namespace-configuration-operator-local charts/namespace-configuration-operator -n namespace-configuration-operator-local --create-namespace
 ```
 
 Delete...
 
 ```shell
-helm delete namespace-configuration-operator-helmchart-test -n namespace-configuration-operator-local
+helm delete namespace-configuration-operator-local -n namespace-configuration-operator-local
 kubectl delete -f charts/namespace-configuration-operator/crds/crds.yaml
 ```
 

config/helmchart/templates/_helpers.tpl

@@ -50,14 +50,3 @@ Selector labels
 app.kubernetes.io/name: {{ include "namespace-configuration-operator.name" . }}
 app.kubernetes.io/instance: {{ .Release.Name }}
 {{- end }}
-
-{{/*
-Create the name of the service account to use
-*/}}
-{{- define "namespace-configuration-operator.serviceAccountName" -}}
-{{- if .Values.serviceAccount.create }}
-{{- default (include "namespace-configuration-operator.fullname" .) .Values.serviceAccount.name }}
-{{- else }}
-{{- default "default" .Values.serviceAccount.name }}
-{{- end }}
-{{- end }}

config/helmchart/templates/manager.yaml

@@ -32,6 +32,18 @@ spec:
         name: {{ .Chart.Name }} 
         resources:
           {{- toYaml .Values.resources | nindent 12 }}
+        livenessProbe:
+          httpGet:
+            path: /healthz
+            port: 8081
+          initialDelaySeconds: 15
+          periodSeconds: 20
+        readinessProbe:
+          httpGet:
+            path: /readyz
+            port: 8081
+          initialDelaySeconds: 5
+          periodSeconds: 10
       {{- with .Values.nodeSelector }}
       nodeSelector:
         {{- toYaml . | nindent 8 }}

config/manager/manager.yaml

@@ -44,9 +44,6 @@ spec:
           initialDelaySeconds: 5
           periodSeconds: 10
         resources:
-          limits:
-            cpu: 100m
-            memory: 30Mi
           requests:
             cpu: 100m
             memory: 20Mi