nextcloud / helm

A community maintained helm chart for deploying Nextcloud on Kubernetes.
GNU Affero General Public License v3.0
334 stars 269 forks source link

add default of "" for externalDatabase.host; update README to be a little more clear #634

Closed jessebot closed 1 month ago

jessebot commented 1 month ago

Description of the change

Adds a default for externalDatabase.host to be "" so it's clear the type taken, and also reminds people that they need to quote when using a : in a value in yaml. Also adds a bit more of a comment for that value and updates the README to reflect both changes.

Benefits

Just resolves tiny bit of possible confusion around how to specify a database port.

Possible drawbacks

none that I can think of

Applicable issues

Checklist