Skip to content

allow uuid being created and managed by kubernetes#73

Closed
colearendt wants to merge 3 commits intoapache:mainfrom
colearendt:close-39
Closed

allow uuid being created and managed by kubernetes#73
colearendt wants to merge 3 commits intoapache:mainfrom
colearendt:close-39

Conversation

@colearendt
Copy link
Contributor

@colearendt colearendt commented May 10, 2022

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.

  • Chart Version bumped
  • e2e tests pass
  • Variables are documented in the README.md
  • Chart tgz added to /docs and index updated

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

A value for couchdbConfig.couchdb.uuid must be set

2 participants