Skip to content

Conversation

@eternity1984
Copy link
Contributor

There was an inconsistency between the Helm chart templates and values.yaml regarding the connection settings for MongoDB and Elasticsearch.

{{- if .Values.mongodb.enabled }}
MONGO_URI: {{ printf "mongodb://%s-mongodb:27017/%s" .Release.Name (include "growi.name" .) }}
{{- else }}
MONGO_URI: {{ .Values.mongodb.uri }}
{{- end }}
{{- if .Values.elasticsearch.enabled }}
ELASTICSEARCH_URI: {{ printf "http://elasticsearch-master:9200/%s" (include "growi.name" .) }}
{{- else }}
ELASTICSEARCH_URI: {{ .Values.elasticsearch.uri }}
{{- end }}

In values.yaml, the keys were defined as url, but the templates referenced uri.
This caused the custom values not to be applied when enabled: false.

Updated values.yaml to use uri instead of url.
Verified that MongoDB and Elasticsearch connections work correctly with the updated keys.

@eternity1984 eternity1984 changed the title chore: rename "url" to "uri" chore: updated values.yaml to use uri instead of url Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant