fix(common): correct some mistakes in configMap handling in common (#1369)

* add test for configmap

* fix runtests and mistakes in configmap handling

* bump

* hmmm

* small fix

charts/library/common/Chart.yaml

@@ -15,4 +15,4 @@ maintainers:
 name: common
 sources: null
 type: library
-version: 8.7.0
+version: 8.7.1

charts/library/common/templates/_postSetup.tpl

@@ -22,6 +22,9 @@ Secondary entrypoint and primary loader for the common chart
     {{- include "common.addon.netshoot" . }}
   {{- end -}}
 
+  {{- /* Build the confimap */ -}}
+  {{ include "common.configmap" . | nindent 0 }}
+
   {{- /* Build the templates */ -}}
   {{- include "common.pvc" . }}
 

charts/library/common/templates/lib/controller/_volumes.tpl

@@ -60,7 +60,7 @@ Volumes included by the controller.
   {{- else if eq $persistence.type "custom" }}
     {{- toYaml $persistence.volumeSpec | nindent 2 }}
   {{- else }}
-    {{- fail (printf "Not a valid persistence.type (%s)" .Values.persistence.type) }}
+    {{- fail (printf "Not a valid persistence.type (%s)" $persistence.type) }}
   {{- end }}
 {{- end }}
 {{- end }}

charts/library/common/values.yaml

@@ -742,7 +742,7 @@ persistence:
   # @default -- See below
   configmap-example:
     enabled: false
-    type: configmap
+    type: configMap
     # -- Specify the name of the configmap object to be mounted
     objectName: myconfig-map
     # -- Where to mount the volume in the main container.