-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Provide helper to set DB password as env variables across products #493
Conversation
This is a great idea. I have not done a code review, but I agree with the change in spirit. A few initial comments:
|
Thanks for taking a look Sam. Good catch on the Connect side, will add those settings (optional if someone wants to use a separate db/schema). It's a good question about whether should we reduce complexity and instead of using rstudio-library helper (this PR) just document in each chart setting the following which will accomplish the same thing.
I would love thoughts from the different product teams. If that approach seems more straightforward can refactor this to a docs things only. |
I am going to close this one and go with a similar approach proposed by @SamEdwardes. After talking with a few folks it seems a more straightforward path that accomplishes the same thing, with the added benefit of showing folks how to set env variables in our helm charts. |
This PR closes: #422 by providing a consistent way to set PostgreSQL database passwords as env variables from K8s secrets in Workbench, Connect and Package Manager. This is now possible since Workbench supports setting the database password as an env variable in the 2024.04.0 release (Connect and Package Manager already supported this). Currently, admins need to either set the password in the values or imperatively during
helm install
. Both are not very k8s native ways of doing things.rstudio-library
contains a_database-env.tpl
helper to create an env variable from a secret. This is then used by each of the product charts.For each product chart, the following values can now be specified:
Package Manager has an additional set of values for the usage database since it is separate:
This PR also includes a common README template. After this is live, we can change the admin guides to use these options, streamlining the install process.