Generic sidecar support

charts/onechart/templates/deployment.yaml

@@ -90,14 +90,16 @@ spec:
           {{- end }}
           resources:
             {{- toYaml .Values.resources | nindent 12 }}
-        {{- if .Values.debugSidecarEnabled }}
+        {{- if .Values.sidecar }}
-        - name: {{ template "robustName" .Release.Name }}-debug
+        - name: {{ template "robustName" .Release.Name }}-sidecar
           securityContext: *securityContext
-          image: {{ .Values.debugSidecar.image }}
+          image: {{ .Values.sidecar.repository }}:{{ .Values.sidecar.tag }}
+          {{- if .Values.sidecar.command }}
           command:
-            - {{ .Values.debugSidecar.shell }}
+            - {{ .Values.sidecar.shell }}
             - -c
-            - {{ .Values.debugSidecar.command | quote }}
+            - {{ .Values.sidecar.command | quote }}
+          {{- end }}
           {{- if or (or (.Values.vars) (.Values.secretEnabled)) .Values.sealedSecrets }}
           envFrom: *envFrom
           {{- end }}

charts/onechart/tests/deployment_sidecar_test.yaml

@@ -3,9 +3,11 @@ templates:
   - deployment.yaml
   - configmap.yaml
 tests:
-  - it: Should add a debug sidecar
+  - it: Should inject a sidecar if one is specified
     set:
-      debugSidecarEnabled: true
+      sidecar:
+        repository: debian
+        tag: stable-slim
     asserts:
       - equal:
           path: spec.template.spec.containers[1].image

charts/onechart/values.yaml

@@ -44,12 +44,6 @@ secretEnabled: false
 
 shell: "/bin/sh"
 
-debugSidecarEnabled: false
-debugSidecar:
-  image: debian:stable-slim
-  shell: "/bin/bash"
-  command: "while true; do sleep 30; done;"
-
 podDisruptionBudgetEnabled: true
 spreadAcrossNodes: false