Skip to content

Add mariadb-cluster helm chart #1275

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

hedgieinsocks
Copy link

@hedgieinsocks hedgieinsocks commented May 11, 2025

Related to #1273

@mmontes11 mmontes11 self-requested a review May 16, 2025 13:43
Copy link
Member

@mmontes11 mmontes11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks a lot for the initiative on this. Planning to release and document this as part of 0.39.0.

Will following up with some adaptations to accomodate this in our CI and release process.

Is it still WIP? Happy to merge if it's not.

@hedgieinsocks hedgieinsocks changed the title [WIP] Add mariadb-cluster helm chart Add mariadb-cluster helm chart May 16, 2025
@hedgieinsocks
Copy link
Author

Well, I'm content with the initial version, so removing WIP. I have not done much of manual e2e testing though to be honest :)

@mmontes11
Copy link
Member

I have not done much of manual e2e testing though to be honest :)

I'm happy enough with the automation you've built. Manual e2e testing will be conducted at the end of the release, planning to automate this at some point covering Helm installation.

Will hold the merge until manual e2e testing is performed, since the chart will be automatically released after merging to main:

if: github.ref == 'refs/heads/main'

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