On Thu, Jan 20, 2022 at 03:38:43PM -0800, Bryce Harrington wrote:
> On Thu, Jan 20, 2022 at 09:42:47AM -0800, Bryce Harrington wrote:
> > On Thu, Jan 20, 2022 at 12:18:34PM +0100, Paul Gevers wrote:
> > > Hi Kunal,
> > >
> > > Thanks for the analysis.
> > >
> > > On 20-01-2022 10:56, Kunal Mehta wrote:
> > > > > I can't tell what it may be trying to encode, but presumably it's
> > > > > either
> > > > > Main_Page or something used by Main_Page, which I'm guessing should
> > > > > only
> > > > > take a fraction of a second to encode. I suppose we could test
> > > > > increasing max_execution_time. But my suspicion is that the encoder
> > > > > is
> > > > > getting stuck in a loop or a recursion.
> > > >
> > > > 30 seconds seems a bit absurd, but given an empty opcache/jit, empty
> > > > caches, on 32-bit VMs, it's within the realm of possibility...I think
> > > > bumping max_execution_time is a good next step. Should the test add a
> > > > php.ini snippet or something?
> > >
> > > I think that would be great. But please upload to Debian as well, let's
> > > not
> > > only have these tests in Ubuntu.
>
> I've kicked off a test with this, to see if adjusting the timeout would
> be worth considering as a possible solution.
Looks like you were right Kunal, increasing the timeout did indeed let
the tests pass with php8.1 on armhf:
https://autopkgtest.ubuntu.com/packages/m/mediawiki/jammy/armhf
https://autopkgtest.ubuntu.com/results/autopkgtest-jammy/jammy/armhf/m/mediawiki/20220121_020236_5c320@/log.gz
https://launchpad.net/ubuntu/+source/mediawiki/1:1.35.5-1ubuntu3
However like I mentioned, I think a more proper fix than what I did
above would be to make the dep8 test set up an .htaccess file with the
setting, rather than patching it in like this. But it does appear that
adjusting the timeout should resolve the issue we're seeing on armhf,
and if the issue Debian has on i386 is also timeout-related may solve it
there too.
Bryce
