GHA: Linux: test only one architecture per Debian version - #10945
GHA: Linux: test only one architecture per Debian version#10945Al2Klimov wants to merge 1 commit into
Conversation
Al2Klimov
added
area/ci
Al2Klimov
left a comment
Al2Klimov
left a comment
There was a problem hiding this comment.
64-bit in general is already covered by other distros.
| - distro: debian:11 | ||
| platform: linux/386 | ||
| - distro: debian:12 | ||
| platform: linux/386 |
There was a problem hiding this comment.
IMAO these are enough for Debian.
|
That's, btw, exactly what we do for ARM64 – we even cover only one Debian version. |
dc8b711 to
3801430
Compare
jschmidt-icinga
left a comment
jschmidt-icinga
left a comment
There was a problem hiding this comment.
Still not sure we should do this honestly. And debian:11 I'd rather remove entirely since it's eol now, at least once it has been dropped from our support matrix.
|
Please name your concerns more precisely. |
There's a lot of architecture specific code paths in boost and potentially the used kernel that may influence the build and/or test. For that reason, if, and as long as we support both x86_32 and x86_64 builds on those platforms, we should also build and test both in the CI. I'd be open to removing 32bit support entirely, or debian:11 entirely, but that's not my decision to make. But I don't think just not testing x86_64 on those distros is a good idea, as long as we still support them, and that is (at the moment) my decision to make because you gave me this PR to review. |
|
Fair.
Won't bring much. Ubuntu 22 has also PHP 8.2 and is still a thing for, IIRC, one more year. |
2 participants
You complained about too many stuff in the GHA. Here's my peace offer.