Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

initdb-configmap does not repsect fullnameOverride #58

Open
snowpoke opened this issue Nov 19, 2024 · 0 comments · May be fixed by #60
Open

initdb-configmap does not repsect fullnameOverride #58

snowpoke opened this issue Nov 19, 2024 · 0 comments · May be fixed by #60

Comments

@snowpoke
Copy link

snowpoke commented Nov 19, 2024

The template for initdb-configmap is currently defined as follows:

{{- if .Values.postgresql.enabled }}
apiVersion: v1
kind: ConfigMap
metadata:
  name: {{ .Release.Name }}-postgresql-initdb
  labels:
  {{ include "matrix-authentication-service.labels" . | nindent 4}}
data:
[...]

This causes difficulties when mas is deployed as a subchart as in the https://github.com/small-hack/matrix-chart project: Both synapse's postgresql and mas's postgresql deployment's initdb ConfigMap base their name on .Release.Name, leading to a naming conflict.

All other resources base their name on the expected matrix-authentication-service.fullname field, but for initdb-configmap, it was changed to .Release.Name in this commit "fix init-db configmap name" (ca81eef), so I imagine this behavior is intentional? I don't understand why this choice was made however.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant