Helm Chart cannot use a database if patroni.enabled=false

[pbolduc]

Describe the bug

When using the Helm chart, if I set patroni.enabled=false, the chart cannot be deployed due to an error. I want to use a different HA database deployed separately from the COMS helm chart. We have a number of databases in our system, I do not want duplicate Patroni clusters which requires more resources and monitoring.

To Reproduce

Steps to reproduce the behavior:

1. Run

helm upgrade coms ./common-object-management-service-0.0.8.tgz --install --values coms-values.yaml --set patroni.enabled=false

2. See error,

Error: template: common-object-management-service/templates/deploymentconfig.yaml:43:27: executing "common-object-management-service/templates/deploymentconfig.yaml" at <include "patroni.fullname" .Subcharts.patroni>: error calling include: template: no template "patroni.fullname" associated with template "gotpl"

Expected behavior

I would be expected if not enabling patroni in the chart, I can reference an existing postgres database with appropriate credentials.

Screenshots

Desktop (please complete the following information):

• OS: [e.g. iOS]
• Browser [e.g. chrome, safari]
• Version [e.g. 22]

Smartphone (please complete the following information):

• Device: [e.g. iPhone6]
• OS: [e.g. iOS8.1]
• Browser [e.g. stock browser, safari]
• Version [e.g. 22]

Additional context

[jujaga]

Thanks for this report. We have plans to support non-Patroni dependency managed databases via the dbSecretOverride value in values.yaml and have added it into our backlog.

[jujaga]

Addressed in #158