helm-charts
helm-charts copied to clipboard
Automatically configure Postgres and MySQL databases on startup.
What was changed
Automatically create databases and apply the schemas to them for Postgres and MySQL.
Why?
In order to be able to create new environments via Terraform without having to manually create the databases between the provisioning of the cluster and the deployment of the helm chart.
Checklist
- How was this tested:
I used this to provision a Postgres cluster, but no other testing has been done. In particular, MySQL hasn't been tested at all.
- Any docs updates needed?
Probably? I'm just giving this back to hopefully help other people, but I don't have the time to do any more work on it myself.
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.
:white_check_mark: rogersd
:x: Dan Rogers
Dan Rogers seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.
Hi team, seems like duplicate of #275. Would love to see one of them merged
Added this patch to our branch, if anyone needs it:
https://github.com/contra/helm-charts/commit/ff0d75f302eb69106468a4d0f6f9bc2a006128a8
I can confirm this works. Can this get merged?
@rogersd I think you need to sign the Contributor License Agreement https://cla-assistant.io/temporalio/helm-charts?pullRequest=281
This PR will close this issue https://github.com/temporalio/helm-charts/issues/349
@rogersd I think you need to sign the Contributor License Agreement https://cla-assistant.io/temporalio/helm-charts?pullRequest=281
I’ve signed it, but I’m not sure if there’s something that needs to be done for GitHub to pick that up?
@jeenadeepak What needs to happen to pass the cla?
Anyway we can help with this being approved?
I'm taking over this PR.
I'm taking over this PR.
Thanks, @mindaugasrukas !
I'm closing this in favor of #405 I had to fix commands, but overall everything is ok.