[GitHub] [superset] NickLarsenNZ commented on issue #24218: Helm chart with default values fails to install

["NickLarsenNZ (via GitHub)" <gi...@apache.org>]

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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org