temporalio / helm-charts

Temporal Helm charts
MIT License
282 stars 316 forks source link

[Feature Request] Allow to skip database creation #440

Closed ascknx closed 1 month ago

ascknx commented 8 months ago

Is your feature request related to a problem? Please describe.

I want to use an existing database with an user that don't have the permission to create the database.

When running the helm chart I get the following error:

Unable to create SQL database.  {"error": "pq: permission denied to create database", "logging-call-at": "handler.go:94"}

Describe the solution you'd like

It would be good to have a flag in the helm chart to switch off the db creation and keep the schema creation active (so setting schema.setup.enabled=false doesnt work for me)

debugger24 commented 8 months ago

I can pick this up.

VLZZZ commented 3 months ago

Hi!

I've shared my workaround in this PR - https://github.com/temporalio/helm-charts/pull/480

However I'm not quite sure on the naming contracts and structure, any concerns are appreciated.