NickLarsenNZ commented on issue #24218: URL: https://github.com/apache/superset/issues/24218#issuecomment-1563557899
This one is a little tricky to fix. The DB host value is set to include the release-name and the chart name: https://github.com/apache/superset/blob/9df8d8dda538095ba3b30d1f82aa81acf17e1c95/helm/superset/values.yaml#L255 I can fix it to only be the `{{ .Release.Name }}`, but then when `namePrefix` is set, the postgres DB service name doesn't include the release name anymore, while the DB Host variable does and doesn't include the `namePrefix`. I could set the `postgresql.primary.fullname` to match the above, but that would be a breaking change for existing deployments. So I think I'll need to make another helper that solves that logic. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
