Merge pull request #59 from cloudkite-io/update-monitoring-vpa

fix(monitoring): ensure vpa mode is quoted & make minReplicas optional

Author: MuriukiPM

monitoring/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: monitoring
 description: Helm chart for Prometheus
 type: application
-version: 0.4.1
+version: 0.4.2
 dependencies:
 - name: kube-prometheus-stack
   version: 69.8.2

monitoring/templates/vpa.yaml

@@ -10,8 +10,10 @@ spec:
     kind:       {{ $vpaAppConfig.kind }}
     name:       {{ if eq $vpaAppName "prometheus" }}{{ $vpaAppName }}-{{ template "monitoring.fullname" $ }}-{{ $vpaAppName }}{{ else }}{{ template "monitoring.fullname" $ }}-{{ $vpaAppName }}{{- end }}
   updatePolicy:
-    updateMode: {{ $vpaAppConfig.mode | default "Auto" }}
-    minReplicas: {{ $vpaAppConfig.minReplicas | default "1" }}
+    updateMode: {{ $vpaAppConfig.mode | default "Off" | quote }}
+    {{ if $vpaAppConfig.minReplicas }}
+    minReplicas: {{ $vpaAppConfig.minReplicas }}
+    {{- end }}
 ---
 {{- end }}
 {{- end }}

monitoring/values.yaml

@@ -125,4 +125,3 @@ vpaApps:
       kind: Deployment
     prometheus-node-exporter:
       kind: DaemonSet
-      minReplicas: 2