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

[bitnami/mariadb] Don't overwrite existing secrets #9267

Closed
wants to merge 2 commits into from
Closed

[bitnami/mariadb] Don't overwrite existing secrets #9267

wants to merge 2 commits into from

Conversation

mamiu
Copy link
Contributor

@mamiu mamiu commented Mar 2, 2022

Description of the change

Don't overwrite already generated secret values on mariadb chart upgrade

Benefits

MariaDB doesn't break on chart upgrade.

Possible drawbacks

None.

Applicable issues

Additional information

Checklist

mamiu added 2 commits March 3, 2022 02:03
Don't overwrite already generated secret values on chart upgrade
Issue #7403 (#7403)

Signed-off-by: Paul Miu <[email protected]>
Don't overwrite already generated secret values on chart upgrade
Issue #7403 (#7403)

Common chart version has to be bumped.

Signed-off-by: Paul Miu <[email protected]>
@carrodher
Copy link
Member

Thanks for creating this PR. Please, note it is needed to sign the commits. Your signature certifies that you wrote the patch or otherwise have the right to contribute the material. You can find more info about this topic, in the “Sign your work” section from the contributing guidelines.

In the same way please note it's needed to create separate PRs per Helm chart, in this case you're modifying bitnami/common and bitnami/mariadb in a single PR

@mamiu
Copy link
Contributor Author

mamiu commented Mar 2, 2022

@carrodher Ok sure, I'll separate them.

@mamiu mamiu closed this Mar 2, 2022
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.

[bitnami/mariadb] Don't overwrite generated secret values on chart upgrade
2 participants