nextcloud / server

☁️ Nextcloud server, a safe home for all your data
https://nextcloud.com
GNU Affero General Public License v3.0
26.67k stars 4k forks source link

Bring back 32 bit support on Nextcloud 26 #36072

Closed PVince81 closed 1 year ago

PVince81 commented 1 year ago

Tasks

Regarding the typing, instead of removing typing like for stable25 we should use the union types available thanks to PHP 8 and use int|float.

Running the CI on 32 bits is not that straightforward because Github does not propose it so one has to run a docker in a docker or something along these lines. Need someone with knowledge on CI/docker.

On Drone, 64bit hardware is likely able to run 32bit code. See "arch" field in https://docs.drone.io/pipeline/configuration/

It should not be too hard to fix once we have proper CI, given we have the PR for stable25 to use as an example of how to fix stuff. That said it seems stable25 is still not passing test suite on 32bits.

Estimate: possibly 2 weeks effort if not more depending on hurdles and whether we need to port Github workflow tests to Drone

@AndyScherzinger @DaphneMuller @come-nc @blizzz

PVince81 commented 1 year ago

@come-nc can you update the status, in case there was progress on the revertings ?

come-nc commented 1 year ago

Only reverted the first one in my PR, I checked the boxes with current PR state.

Current CI status in the PR: Tests: 14429, Assertions: 37379, Errors: 8, Failures: 6, Warnings: 29, Skipped: 42.

On the 8 errors:

On the 6 failures:

come-nc commented 1 year ago
szaimen commented 1 year ago

Looks like everything done now or is something missing?

blizzz commented 1 year ago

Looks like everything done now or is something missing?

I am not aware of anything missing. @come-nc can you confirm?

come-nc commented 1 year ago

Backport of CI and fixes to 25 is missing but that will have to wait 25.0.5.

blizzz commented 1 year ago

Backport of CI and fixes to 25 is missing but that will have to wait 25.0.5.

which ones did i miss? (but this issue is about 26?)

come-nc commented 1 year ago

Backport of CI and fixes to 25 is missing but that will have to wait 25.0.5.

which ones did i miss? (but this issue is about 26?)

The fixes which were done in the PR adding CI: https://github.com/nextcloud/server/pull/36120 The CI needs to be backported to 25 and some things will break, and some of the fixes will have to be backported. But it’s not an easy backport because typing is different on 25 and 26.

ZachHandley commented 1 month ago

hey so, I don't know if this is the right issue, but I have this error

image

only on files over 2 GB, these aren't four, they're 3.7 GB, so it's extra weird. From the research I've done, it's related to #16431 -- if I'm wrong please don't re-open, but I can't upload anything and I have a hetzner cloud 32 bit so.

AndyScherzinger commented 1 month ago

Can't say for sure @ZachHandley - best to raise a new issue so folks can reply to it. I know there is a limitation in terms of file size 32Bit can handle due to the limitations that come with it within PHP.

sunjam commented 1 month ago

hey so, I don't know if this is the right issue, but I have this error image only on files over 2 GB, these aren't four, they're 3.7 GB, so it's extra weird. From the research I've done, it's related to #16431 -- if I'm wrong please don't re-open, but I can't upload anything and I have a hetzner cloud 32 bit s

Try asking Hetzner assistance directly. Good luck with it.