-
Notifications
You must be signed in to change notification settings - Fork 403
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
Update Nextcloud's MariaDB container safely #1682
base: master
Are you sure you want to change the base?
Conversation
🎉 Linting finished with no errors or warnings 🎉Thank you for your submission! This is an automated linter that checks for common issues in pull requests to the Umbrel App Store. Please review the linting results below and make any necessary changes to your submission. Linting Results
Legend
|
TODO: upgrade MariaDB #1547 (comment):
|
I just wanted to check in and ask if you have any rough estimate on when it might be ready. I know you have a lot on your plate, so no rush - I really appreciate all the hard work you’re putting into this. Thanks again for your efforts! 👌 |
Hey @dennysubke, sorry for the delay on this. I need to carve out some significant time for this single app update. The issue here is that we'll need to test that whatever method we come up with for updating the database will work for users who are updating through multiple major versions (e.g., someone who is still on v28.x needing to get to v30.0.1 via I haven't given it a ton of thought so far, but one thing we need to do when changing the mariadb image is have the app start up with the cc @sharknoon |
Thank you for the detailed explanation! This sounds like a well-thought-out approach. Of course, safety comes first! 👍 |
Also to consider for this update: #1771 |
Any news about this? Now v30.0.4 is out. Both v30.0.2 and v30.0.3 releases contain security fixes. |
@nmfretz 👋🏻 |
Thanks for the ping @dennysubke. In order to not delay Nextcloud updates I have updated to 30.0.4 in #1997, and have renamed this PR to focus only on the database update. |
Thank you for your great work! 🙏🏻 |
🤖 This is an automated pull request to update nextcloud to version v30.0.1. This PR must be reviewed and tested before merging.