fix(chart): critical reliability and templating fixes (#180)

Signed-off-by: Philippe Noël <philippemnoel@gmail.com>

Author: philippemnoel

charts/cluster/templates/_barman_object_store.tpl

@@ -5,9 +5,11 @@
 {{- end }}
 
 {{- if or (.scope.endpointCA.create) (.scope.endpointCA.name) }}
+  {{- $endpointCAName := coalesce .scope.endpointCA.name (printf "%s-ca-bundle" .chartFullname) }}
+  {{- $endpointCAKey := default "ca-bundle.crt" .scope.endpointCA.key }}
   endpointCA:
-    name: {{ .scope.endpointCA.name }}
-    key: {{ .scope.endpointCA.key }}
+    name: {{ $endpointCAName }}
+    key: {{ $endpointCAKey }}
 {{- end }}
 
 {{- if .scope.destinationPath }}

charts/cluster/templates/databases.yaml

@@ -1,4 +1,5 @@
 {{- range .Values.databases }}
+{{- $dbOwner := default .name .owner }}
 ---
 apiVersion: postgresql.cnpg.io/v1
 kind: Database
@@ -19,18 +20,18 @@ spec:
   cluster:
     name: {{ include "cluster.fullname" $ }}
   ensure: {{ .ensure | default "present" }}
-  owner: {{ .owner }}
+  owner: {{ $dbOwner }}
   template: {{ .template | default "template1" }}
   encoding: {{ .encoding | default "UTF8" }}
   databaseReclaimPolicy: {{ .databaseReclaimPolicy | default "retain" }}
-  {{- with .isTemplate }}
-  isTemplate: {{ . }}
+  {{- if hasKey . "isTemplate" }}
+  isTemplate: {{ .isTemplate }}
   {{- end }}
-  {{- with .allowConnections }}
-  allowConnections: {{ . }}
+  {{- if hasKey . "allowConnections" }}
+  allowConnections: {{ .allowConnections }}
   {{- end }}
-  {{- with .connectionLimit }}
-  connectionLimit: {{ . }}
+  {{- if hasKey . "connectionLimit" }}
+  connectionLimit: {{ .connectionLimit }}
   {{- end }}
   {{- with .tablespace }}
   tablespace: {{ . }}
@@ -63,7 +64,7 @@ spec:
   schemas:
   {{- range . }}
     - name: {{ .name }}
-      owner: {{ .owner }}
+      owner: {{ default $dbOwner .owner }}
       ensure: {{ .ensure | default "present" }}
   {{- end }}
   {{- end }}