-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
patch the version file to allow upgrading from owncloud > v10.13 #56791
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
patch the version file to allow upgrading from owncloud > v10.13 #56791
Conversation
szaimen
left a comment
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.
LGTM but did not test
| ], | ||
| 'owncloud' => [ | ||
| '10.13' => true, | ||
| '10.14' => true, |
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 thought 10.14 does not exist?
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.
It does - but never saw it in the wild...
|
@binford6000 please check the failing CI. Seems to be related |
Some customers already migrated from 10.15.3 and needed to edit the version.php file before the migration. 10.16 was recently published and it seems migration from there also works fine. Signed-off-by: Sebastian Möbus <[email protected]>
Signed-off-by: Andy Scherzinger <[email protected]>
3b5412a to
f562053
Compare
|
The branch name is invalid, at least on windows... so I can't fix the commit messages |
Same for me unfortunately |
|
Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22 |
|
I think this needs some backporting to 25, no? |
|
/backport to stable27 |
|
/backport to stable26 |
|
/backport to stable25 |
|
backports matching #40977 |
|
Docs PR nextcloud/documentation#13910 |
|
@AndyScherzinger I am not sure about the backports. Peobably backporting to nc32 for now is enough? Wdyt @binford6000 ? |
Usually our customers want to migrate to latest or latest-1. So Backporting to 31 would be sufficient in my opinion. |
But see the docs, wouldn't you update to latest v10 cO, than v25 Nc (due to the PHP requirements) and than upgrade Nc versions from there? |
Sure, but looking at my last big migrations people turn on maintenance mode, stop webserver/php, bump php to 8.3 and then start occ upgrade with NC31/32 directly which definitely is proven to work (ETH Zurich, upcoming SWITCHdrive and others). One could say that our documentation would need to be updated as well... |
|
/backport to stable32 |
|
/backport to stable31 |
|
/backport to stable25 |
I guess 26-30 is needed in that case as well? |
Some customers already migrated from 10.15.3 and needed to edit the version.php file before the migration. 10.16 was recently published and it seems migration from there also works fine.
nextcloud/documentation#13910
Summary
TODO
Checklist
3. to review, feature component)stable32)