-
Notifications
You must be signed in to change notification settings - Fork 269
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
Upgrade Redis, Postgresql, and Maridb to latest OCI Bitnami helm chart versions #420
Conversation
- Upgrades all bitnami subcharts to the latest major versions at time of commit - Updates to new preferred OCI-based container registry Signed-off-by: JesseBot <[email protected]>
Signed-off-by: jessebot <[email protected]>
1330141
to
effdad9
Compare
@@ -1,6 +1,6 @@ | |||
apiVersion: v2 | |||
name: nextcloud | |||
version: 3.5.19 | |||
version: 4.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version: 4.0.0 | |
version: 3.6.0 |
This can also be 3.6.0
if we only want to bump a minor version of this chart.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would say that since the subchart updates are major, I'd make it a major release.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
As new bitnami charts now support Thank you for your work @jessebot |
Signed-off-by: JesseBot <[email protected]>
Sorry for the delay! Longhorn failed for me recently (long story, but they pushed up a breaking change that had cascading effects and took out my whole home lab and backups D:) I've been rebuilding my home lab (and recovering backups), and should probably be done in another day. In the meantime, if others have tested upgrade this, I'm happy to see it merged 💙 |
This version installs seamlessly on my RPi4, with prepackaged mariadb. Not an upgrade though, as this looks like the first ARM64 compatible release, I only proceed to a fresh install. |
Just tried a fresh install on 4*bananapi CM4(arm64) with external postgres db, works perfectly👍 |
Signed-off-by: JesseBot <[email protected]>
@provokateurin I still haven't tested the upgrade, but since two others have tested fresh installs, do we want to merge this anyway? Unfortunately my homelab is still out of commission due to some other issues in my current stack (dealing with some issues lower in the stack with secrets management), so it'll probably be at least another few days till I can start testing nextcloud via helm specifically again. We could just make sure the release notes have a link to the docs for upgrades for postgresql and to please report any other steps you had to do directly to us so we can inform others? I feel bad not merging this :< |
Sounds like a good plan |
…t versions (nextcloud#420) * Upgrade redis, postgresql, and maridb sub charts to latest - Upgrades all bitnami subcharts to the latest major versions at time of commit - Updates to new preferred OCI-based container registry Signed-off-by: JesseBot <[email protected]> * update Chart.lock file for subchart updates Signed-off-by: jessebot <[email protected]> --------- Signed-off-by: JesseBot <[email protected]> Signed-off-by: jessebot <[email protected]>
Pull Request
Description of the change
14.4.x
to15.3.0
10.6.x
to10.11.4
6.2.x
to7.0.12
Benefits
This puts us at the latest versions, so we aren't deploying a major version out of date. I haven't checked, but there may also be security patches that we would benefit from here.
Possible drawbacks
postgresql
bitnami sub-chart will need to upgrade their databases to stay current according to Bitnami's postgresql chart README . Official upgrade instructions can be found here.Applicable issues
Don't think there's any right now.
Additional information
Bumping major version of chart to account for major version bump of subcharts, but we can also do a minor version here as all of these are optional? Open to feedback on which number to bump.
Upgrade of Redis should not incur any extra work according to their README docs here.
MariaDB doesn't have a major appVersion bump so there's no upgrade instructions listed.
Checklist
Chart.yaml
according to semver.