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

Update nextcloud app version to 27.0.1 #419

Merged
merged 3 commits into from
Jul 28, 2023
Merged

Conversation

jessebot
Copy link
Collaborator

Pull Request

Description of the change

Updates the default image tag to 27.0.1

Benefits

latest stable version as of yesterday

Possible drawbacks / Considerations

Still no upgrade guide. This'll be the first time I'm upgrading a production instance, so happy to write up a guide on it. Is it just setting nextcloud.update to 1? 🤔 Does that enable the auto-updater? Right now I have it set to the default, 0 and when I go to https://cloud.mydomain.com/updater/ it gives a 404, so maybe that's why? If I set it to 1 before upgrading the helm chart version, and just run a helm upgrade with the new values.yaml, will that break anything? I'll obviously take backups before this, but it would be nice to have a section of the README for this as upgrades are a common question we get in the issues.

Checklist

@jessebot
Copy link
Collaborator Author

jessebot commented Jul 23, 2023

oh, the 404 was because you're not supposed to upgrade that way. 🤦 ref:

@jessebot
Copy link
Collaborator Author

Tested the upgrade by only ticking up image.tag to 27.0.1-fpm (fpm because I'm using nginx) and then just syncing my nextcloud Argo CD app which uses this helm chart, and everything went ok except the Nextcloud News app, which isn't compatible with 27.0.1 yet. There were also some missing indices, but that was easily solved by running:

# where NEXTCLOUD_POD is *your* nextcloud pod
kubectl exec -it $NEXTCLOUD_POD -- su -s /bin/bash www-data -c "php occ db:add-missing-indices"

Maybe I should try to get the release notes file into this release so we can mention this in the release notes? 🤔 Either way, I will add an "Upgrading" section to the README and note that the web updater doesn't work with the docker image and to instead please just update the docker tag (after doing a backup). Perhaps a short backup section might also make sense. I'm currently using k8up and longhorn for my backups, but I'm sure velero users would like to contribute to that down the line as well, and backups are essential before an upgrade.

@@ -1,7 +1,7 @@
apiVersion: v2
name: nextcloud
version: 3.5.19
appVersion: 27.0.0
version: 3.5.20
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
version: 3.5.20
version: 3.6.1

This can also be 3.6.1 depending on if #420 get merged before this one or not.

@jessebot jessebot force-pushed the update-nextcloud-app-version branch from 769cc31 to 7cbe6ea Compare July 24, 2023 15:18
@jessebot jessebot merged commit 643417d into main Jul 28, 2023
2 checks passed
@delete-merged-branch delete-merged-branch bot deleted the update-nextcloud-app-version branch July 28, 2023 07:30
benyanke pushed a commit to benyanke/helm that referenced this pull request Aug 18, 2023
* updating nextcloud app version to 27.0.1

Signed-off-by: jessebot <[email protected]>

* adding some docs on upgrades and backups

Signed-off-by: jessebot <[email protected]>

* Update charts/nextcloud/README.md -  add backups link

Signed-off-by: JesseBot <[email protected]>

---------

Signed-off-by: jessebot <[email protected]>
Signed-off-by: JesseBot <[email protected]>
Signed-off-by: Ben Yanke <[email protected]>
raynay-r pushed a commit to raynay-r/nextcloud-helm that referenced this pull request Apr 16, 2024
* updating nextcloud app version to 27.0.1

Signed-off-by: jessebot <[email protected]>

* adding some docs on upgrades and backups

Signed-off-by: jessebot <[email protected]>

* Update charts/nextcloud/README.md -  add backups link

Signed-off-by: JesseBot <[email protected]>

---------

Signed-off-by: jessebot <[email protected]>
Signed-off-by: JesseBot <[email protected]>
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.

2 participants