Replaced part of backup sidecar template with template function from subchart (specifically

osql.envvars) which should mean variable naming mismatch shouldn't happen there anymore

templates/_sidecars.tpl

@@ -21,15 +21,7 @@
         name: {{ .Release.Name }}-vault-snapshot-config
         key: port
   {{- end }}
-  {{- if eq .Values.nosql.type "mongodb" }}
-  # NoSQL storage related environment variables
-  # Note, we only worry about self-hosted options as cloud-based should have their own backups etc...
-  - name: STORAGE_ACCOUNT_CONNECTION_STRING
-    valueFrom:
-      secretKeyRef:
-        name: {{ .Release.Name }}-mongo-credentials
-        key: connection-string
-  {{- end }}
+  {{- include "nosql.envVars" . | nindent 2 }}{{- include "nosql.envVars" . | nindent 2 }}
   # Redis is used for BullMQ, which is how we schedule backups
   # We use this instead of, for instance cron jobs, as it lets us deal with failures
   {{- include "cache.envVars" . | nindent 2 }}