Skip to content

Exchange MariaDB Bitnami Helm Chart with MariaDB Community Operator - #57

Open
MartinWeise wants to merge 5 commits into
mainfrom
feature/mariadb-operator
Open

Exchange MariaDB Bitnami Helm Chart with MariaDB Community Operator#57
MartinWeise wants to merge 5 commits into
mainfrom
feature/mariadb-operator

Conversation

@MartinWeise

Copy link
Copy Markdown
Contributor

Description

This PR implements #56 by replacing the Bitnami MariaDB Galera Chart with the MariaDB Community Operator. This is expected to benefit long-term stability of the Metadata/Data-DBs and increase cloud-nativeness of the deployment.

This PR also shifts both Metadata/Data-DBs from a Multi-Master architecture to a more resilient Primary/Read-Replica architecture (=only 1 master).

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes, no API changes)
  • Dependency update

Related Issue

Closes #56

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Additional Notes

None.

- Remove the Bitnami Galera Chart
- Add the MariaDB Community Operator

Signed-off-by: Martin Weise <martin.weise@tuwien.ac.at>
Signed-off-by: Martin Weise <martin.weise@tuwien.ac.at>
Signed-off-by: Martin Weise <martin.weise@tuwien.ac.at>
@MartinWeise MartinWeise linked an issue Jun 21, 2026 that may be closed by this pull request

@maxlastglance maxlastglance left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Helm lint/template läuft durch, also auf YAML-Ebene wirkt es nicht kaputt, aber ich glaube da sind ein paar Runtime-Themen drin, die wir vor dem Merge fixen sollten.

Hauptpunkt: Ich sehe nicht, dass für 'data-db' die 'dbrepo' Datenbank noch angelegt wird. Es gibt ein Database CR für metadata-db, aber kein entsprechendes für data-db, während der readonly Grant weiterhin auf database: "dbrepo" zeigt. Ich würde erwarten, dass der Grant dann fehlschlägt oder hängen bleibt, außer der Operator legt die DB implizit an.

Außerdem verwenden die SQL Jobs in metadata-db-sqljob.yaml zwar metadata-db, aber als User.Values.datadb.rootUser.user. Mit den Defaults geht das, weil beides root ist, aber sobald jemand den Metadata-DB-User separat überschreibt, bricht es. Das sollte vermutlich .Values.metadatadb.rootUser.user sein, oder?

Signed-off-by: Martin Weise <martin.weise@tuwien.ac.at>
@MartinWeise

Copy link
Copy Markdown
Contributor Author

Hauptpunkt: Ich sehe nicht, dass für 'data-db' die 'dbrepo' Datenbank noch angelegt wird.

I added the database

Außerdem verwenden die SQL Jobs in metadata-db-sqljob.yaml zwar metadata-db, aber als User.

Thanks, I fixed that so it uses the correct metadatadb.rootUser, I also fixed the template for spec.database to actually use the Values in values.yaml.

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.

Replace MariaDB Galera Bitnami-Chart

2 participants