apache / couchdb-helm

Apache CouchDB Helm Chart
https://couchdb.apache.org/
Apache License 2.0
49 stars 64 forks source link

allow uuid being created and managed by kubernetes #73

Open colearendt opened 2 years ago

colearendt commented 2 years ago

What this PR does / why we need it:

We utilize Helm's lookup command to store a generated uuid in an "internal" secret in Kubernetes. This allows generating the uuid, making it persistent, and notifying the user (in NOTES.txt) that this auto-generation happened. We also tell the user how to disable the message by making that value persistent in values.

Which issue this PR fixes

Special notes for your reviewer:

Happy to discuss this approach, messaging, etc. We have found similar patterns to strike a decent balance when secrets / inputs must exist outside of the application.

Open question: should we allow dangerRegenerateAutomatedValues?

(will walk through checklist once validated as a decent approach)

TODO: update docs accordingly / remove the uuid generation

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.