Skip to content
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

Feature: Add support for mysql #74

Merged
merged 13 commits into from
Mar 27, 2024
Merged

Conversation

CASL0
Copy link
Contributor

@CASL0 CASL0 commented Jan 15, 2024

Hi , I've created a pull request that addresses #15 in this project

I have added support for connecting to bundled subchart's MySQL or an external MySQL database.

Could someone please review it and provide feedback?

@jessebot
Copy link
Collaborator

Sorry for such a delay. let me take a look :)

@jessebot
Copy link
Collaborator

This looks great, but we need to bump the helm chart version :) I can merge this for now, and then I'll manually bump the helm chart version. It may also make sense to make the initdb configmap only deploy if mysql is enabled. To unblock though, I'll merge this now. Thanks for your contribution. Please note: I am moving this to my github org: small-hack, but it is also an AGPL focused org of just a few people and will be better maintained with renovate bot there.

@jessebot jessebot merged commit 4a11308 into small-hack:main Mar 27, 2024
@jessebot
Copy link
Collaborator

This feature went out in #78
MySQL was then updated to v9.23.0 in: #80
There is also a major version bump coming in: #85

Thanks, @CASL0 and let me know if anything else needs updating or fixing :)

@CASL0 CASL0 deleted the feature/mysql branch March 28, 2024 01:41
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.

2 participants