-
Notifications
You must be signed in to change notification settings - Fork 706
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
Adding the missing externalDatabase.host and externalDatabase.port fields #7743
base: main
Are you sure you want to change the base?
Adding the missing externalDatabase.host and externalDatabase.port fields #7743
Conversation
…elds from the value.yaml
@brunomedeirosdedalus, you must sign our contributor license agreement before your changes are merged. Click here to sign the agreement. If you are a VMware employee, read this for further instruction. |
✅ Deploy Preview for kubeapps-dev canceled.Built without sensitive environment variables
|
@brunomedeirosdedalus, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
There is a lot room for improving here, like adding under externalDatabase, the DB username, name and so on |
@brunomedeirosdedalus, VMware has approved your signed contributor license agreement. |
bump |
Adding the missing externalDatabase.host and externalDatabase.port fields for values.yaml
Description of the change
Benefits
Possible drawbacks
Applicable issues
Additional information
This PR adds to the values.yaml the missing 'externalDatabase.host' and 'externalDatabase'.port' fields needed to setup and external database.