From bfb8f8ef23fe071f9d5f1412b9202682e2966bfd Mon Sep 17 00:00:00 2001
From: Alan Bridgeman <a.bridgeman@hotmail.com>
Date: Thu, 25 Dec 2025 13:21:15 -0600
Subject: [PATCH] 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 | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/templates/_sidecars.tpl b/templates/_sidecars.tpl
index ab8a1c3..e4a9ad7 100644
--- a/templates/_sidecars.tpl
+++ b/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 }}