helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

Automatically configure Postgres and MySQL databases on startup.

Open rogersd opened this issue 3 years ago • 2 comments

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

  1. 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.

  1. 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.

rogersd avatar Mar 21 '22 16:03 rogersd

CLA assistant check
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.

CLAassistant avatar Mar 21 '22 16:03 CLAassistant

Hi team, seems like duplicate of #275. Would love to see one of them merged

franklinkim avatar Apr 25 '22 06:04 franklinkim

Added this patch to our branch, if anyone needs it:

https://github.com/contra/helm-charts/commit/ff0d75f302eb69106468a4d0f6f9bc2a006128a8

gajus avatar Jan 19 '23 19:01 gajus

I can confirm this works. Can this get merged?

NoUseFreak avatar May 11 '23 11:05 NoUseFreak

@rogersd I think you need to sign the Contributor License Agreement https://cla-assistant.io/temporalio/helm-charts?pullRequest=281

NoUseFreak avatar May 11 '23 11:05 NoUseFreak

This PR will close this issue https://github.com/temporalio/helm-charts/issues/349

medamineamara avatar May 16 '23 21:05 medamineamara

@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?

rogersd avatar May 16 '23 23:05 rogersd

@jeenadeepak What needs to happen to pass the cla?

NoUseFreak avatar Jun 03 '23 19:06 NoUseFreak

Anyway we can help with this being approved?

foogazy avatar Jun 04 '23 08:06 foogazy

I'm taking over this PR.

mindaugasrukas avatar Jul 05 '23 17:07 mindaugasrukas

I'm taking over this PR.

Thanks, @mindaugasrukas !

rogersd avatar Jul 05 '23 20:07 rogersd

I'm closing this in favor of #405 I had to fix commands, but overall everything is ok.

mindaugasrukas avatar Jul 05 '23 21:07 mindaugasrukas