fixes for render checks

Author: kanwarujjaval

charts/countly-migration/templates/_helpers.tpl

@@ -101,7 +101,7 @@ Bundled mode: construct from sibling countly-mongodb chart DNS.
 {{- $host := $bs.host | default (printf "%s-mongodb-svc.%s.svc.cluster.local" $prefix ($bs.namespace | default "mongodb")) -}}
 {{- $port := $bs.port | default "27017" -}}
 {{- $user := $bs.username | default "app" -}}
-{{- $pass := required "backingServices.mongodb.password is required when mode=bundled" $bs.password -}}
+{{- $pass := $bs.password | default "" -}}
 {{- $db := $bs.database | default "admin" -}}
 {{- $rs := $bs.replicaSet | default (printf "%s-mongodb" $prefix) -}}
 mongodb://{{ $user }}:{{ $pass }}@{{ $host }}:{{ $port }}/{{ $db }}?replicaSet={{ $rs }}&ssl=false

charts/countly-migration/templates/secret.yaml

@@ -16,7 +16,15 @@ type: Opaque
 data:
   {{- $secretName := include "countly-migration.fullname" . }}
   {{- $existing := lookup "v1" "Secret" .Release.Namespace $secretName }}
+  {{- if and (not .Values.backingServices.mongodb.password) (eq (.Values.backingServices.mongodb.mode | default "bundled") "bundled") }}
+  {{- if and $existing (index $existing.data "MONGO_URI") }}
+  MONGO_URI: {{ index $existing.data "MONGO_URI" }}
+  {{- else }}
+  {{- fail "backingServices.mongodb.password is required on first install when mode=bundled. Set the value or provide secrets.existingSecret." }}
+  {{- end }}
+  {{- else }}
   MONGO_URI: {{ include "countly-migration.mongoUri" . | b64enc }}
+  {{- end }}
   CLICKHOUSE_URL: {{ include "countly-migration.clickhouseUrl" . | b64enc }}
   {{- if .Values.backingServices.clickhouse.password }}
   CLICKHOUSE_PASSWORD: {{ .Values.backingServices.clickhouse.password | b64enc }}