Add mariadb.global.defaultStorageClass to our values.yaml and clean up the README to also include mariadb.primary.persistence.storageClass and mariadb.primary.persistence.existingClaim as they'd be common parameters.
Benefits
Less confusion about how to use the MariaDB subchart because it better reflects the bitnami helm chart.
Possible drawbacks
none that I can think of, but always happy for feedback :)
Description of the change
Add
mariadb.global.defaultStorageClassto our values.yaml and clean up the README to also includemariadb.primary.persistence.storageClassandmariadb.primary.persistence.existingClaimas they'd be common parameters.Benefits
Less confusion about how to use the MariaDB subchart because it better reflects the bitnami helm chart.
Possible drawbacks
none that I can think of, but always happy for feedback :)
Applicable issues
Additional information
nope
Checklist
Chart.yamlaccording to semver.