Skip to content

Conversation

@ericgaspar
Copy link
Member

@ericgaspar ericgaspar commented Dec 10, 2022

  • Update to 25.0.3
  • Only warn about 32-bit deprecation
  • Update to recommanded PHP8.1

@ericgaspar
Copy link
Member Author

!testme

@yunohost-bot
Copy link
Contributor

Fingers crossed!
Test Badge

@lapineige
Copy link
Member

lapineige commented Dec 10, 2022

Please merge #533 first

tituspijean
tituspijean previously approved these changes Dec 10, 2022
@lapineige
Copy link
Member

Upgrade didn't work fine on my side as it triggered an internal server error, and the log still shows the same error :(

@lapineige
Copy link
Member

That's a quota issue, see: nextcloud/server#34905 (comment)

@lapineige
Copy link
Member

Fixed by this manual change : nextcloud/server#34905 (comment)

@lapineige
Copy link
Member

Let's merge ?

@ericgaspar
Copy link
Member Author

@lapineige It is not extra clear to me if user with 32 bits arch machines will still have to manually apply this tweaks nextcloud/server#34905 (comment)

@lapineige
Copy link
Member

Same for me. I asked on their repository.

Lots of people are asking for this upgrade on the forum, that's why I was suggesting to merge… but maybe it's better to make sure they won't have a broken Nextcloud first 😅

@lapineige
Copy link
Member

lapineige commented Dec 20, 2022

PR ongoing on Nextcloud side to fix that bug nextcloud/server#35734 (update: it need testers).
Let's wait for it ?

And maybe #530 ?

@ericgaspar
Copy link
Member Author

ericgaspar commented Jan 22, 2023

Well yes, the idea is to keep 32-bit compatibility as much as possible. We also need to inform 32-bit user that support will soon be dropped.
How soon is soon? It doesn't seem clear upstream
@Gredin67 If you have a better warning formulation please change it

@lapineige
Copy link
Member

@Gredin67 If you have a better warning formulation please change it

Nextcloud might drop 32bit support in the future. It is recommended that you upgrade to a 64 bits architecture.

This would not add to specific detail about a version number that could fluctuate/be outdated very soon, not urge too much our users to upgrade, and still warn then that they should plan it somewhere in the quite near future.
What do you think ?

lapineige
lapineige previously approved these changes Jan 23, 2023
@lapineige
Copy link
Member

nextcloud/server#35734 (comment)
25.0.3 should be fixing the bug, at least most of the time, so I would vote for a merge. That would release the burden of many people not able to upgrade because of the previous (blocking) warning (and also lighten our support work), and for the few that might still encounter the bug there is a manual fix available.
Last CI and then we merge ?

@lapineige
Copy link
Member

!testme

@yunohost-bot
Copy link
Contributor

Alrighty!
Test Badge

@lapineige
Copy link
Member

Oh and let's merge #530 too ?

@lapineige
Copy link
Member

lapineige commented Jan 23, 2023

Upgrade went fine on my side.
But I still have the quota bug :/

then
ynh_die --message="Sorry, Nextcloud has deprecated 32-bit support"
ynh_print_warn --message="Nextcloud will soon deprecate 32-bit support. It is recommended to upgrade to a 64-bit architecture."
fi
Copy link
Contributor

@Gredin67 Gredin67 Jan 24, 2023

Choose a reason for hiding this comment

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

My suggestion: "Nextcloud will deprecate 32-bit support within the next years. You should start thinking about upgrading to a 64-bit architecture."

@lapineige
Copy link
Member

!testme

@yunohost-bot
Copy link
Contributor

Fingers crossed!
Test Badge

@lapineige
Copy link
Member

Install failed for some strange reason 🤔

@ericgaspar
Copy link
Member Author

This is just the CI failing a successfully installed app

@lapineige
Copy link
Member

CI is fully happy.

Let's merge ?

Copy link
Member

@tituspijean tituspijean left a comment

Choose a reason for hiding this comment

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

Tested and working!

@ericgaspar ericgaspar merged commit a3ef6ef into master Jan 25, 2023
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.

7 participants