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

Merged
merged 1 commit into from
Jul 16, 2025

Conversation

hedgieinsocks
Copy link
Contributor

@hedgieinsocks hedgieinsocks commented May 11, 2025

The initial version of mariadb-cluster helm chart will be able to template:

  • 1 MariaDB
  • list of Database
  • list of User
  • list of Grant
  • list of Backup

Other resources might be included via prs in the future.

closes: #1273

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
Contributor 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'

@mmontes11
Copy link
Member

This will be included in 25.08.0 after I make the adaptations mentioned above.

@Amir-Ghojavand
Copy link

Hi team

Thank you for inviting me to share feedback!

I think it would be valuable to include a backup system, even in the first version of the Helm chart. Backups are critical for production use, and having even a basic option (e.g., scheduled backups to S3 or local PVC) would greatly help users adopt the chart safely from the start.

Happy to discuss this further or help implement it if needed!

Thanks again for your hard work on this project.

If you want, I can also help you:
Draft a technical breakdown of a lightweight S3 backup implementation for MariaDB Helm charts.

@mmontes11
Copy link
Member

@Amir-Ghojavand FYI we have supported PhysicalBackups in this release. They have been merged to main but not released yet. They are documented here:
https://github.com/mariadb-operator/mariadb-operator/blob/release-25.08/docs/PHYSICAL_BACKUP.md

@Amir-Ghojavand
Copy link

no i mean backup for the helm chart not the crd of pbc

@hedgieinsocks
Copy link
Contributor Author

@Amir-Ghojavand I've added backups templating with the corresponding test. Backup is a secondary resource like db/user/grant, so we will not impose it by default. Yet, a consumer will be able to configure it if need be.

@mmontes11 I don't remember saying it, but I kinda intend to leave Chart.yaml (at least versions lol) and helm-docs stuff to you so you could polish it up to your liking and vision.

@mmontes11
Copy link
Member

mmontes11 commented Jul 15, 2025

I don't remember saying it, but I kinda intend to leave Chart.yaml (at least versions lol) and helm-docs stuff to you so you could polish it up to your liking and vision.

Sure thing, will take care of the CI related stuff before the release. Will also add support for PhysicalBackup, the same way you did for Backup

@mmontes11 mmontes11 force-pushed the add/mariadb-cluster branch from 6abfd67 to b9b75c7 Compare July 16, 2025 10:48
@mmontes11 mmontes11 merged commit 54aa93a into mariadb-operator:main Jul 16, 2025
9 checks passed
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.

[Feature] Helm chart for deploying mariadb clusters
3 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy