Bug#1028041: php-excimer: FTBFS on mipsel

2023-02-02 Thread Adrian Bunk
On Thu, Feb 02, 2023 at 02:23:21PM -0500, Kunal Mehta wrote:
> severity 1028041 normal
> thanks
> 
> Hi,
> 
> On 1/6/23 13:47, Adrian Bunk wrote:
> > On Fri, Jan 06, 2023 at 08:59:08AM +0100, Paul Gevers wrote:
> > > =
> > > FAILED TEST SUMMARY
> > > -
> > > ExcimerProfiler CPU profile [tests/cpu.phpt]
> > > =
> > > 
> > > Please fix ASAP to not block the php8.2 transition.
> > 
> > It built after I gave it back to build on the right buildd,
> > so it's now rebuilt with PHP 8.2.
> 
> Thanks.
> 
> > The issue might depend on buildd speed, or some weird difference
> > between buildds.
> 
> Looking through my email, it's flaked before in the past (#1014801). The
> test in question[1] has the following comment:
> 
> // Test aggregateByFunction
> // Typically the parent functions foo() and bar() will have self=0 and
> // inclusive ~= 30. The other 4 functions will have a count of about 30/4 =
> 7.5.
> // The probability of C::member() or baz() having a count of zero is about 1
> in 5600.
> 
> Maybe the known flakiness is worse due to something on the mipsel build?
>...

I am not convinced that this is flakiness, my guess (that could be wrong!)
would be tests failing on all except the fastest mipsel buildds.

It is problematic that the test doesn't output what exactly fails.

On the porterbox the test passes initially, but it fails after
  -$profiler->setPeriod(0.1);
  +$profiler->setPeriod(10);
I do not know whether or not this is the same failure as on the mipsel 
buildds.

> -- Kunal

cu
Adrian



Bug#1028041: php-excimer: FTBFS on mipsel

2023-02-02 Thread Paul Gevers

Hi Kunal,

On 02-02-2023 20:23, Kunal Mehta wrote:
I'm lowering the severity because it no longer blocks the 8.2 transition 
(please revert me if I'm wrong on that).


In my opinion flaky builds and flaky tests are bad because they cost 
quite some time of people that are not involved in the package. E.g. in 
this case Release Team, but imagine a security upload where the build 
fails. But you have flaky and flaky. https://bugs.debian.org/844264 is 
an interesting read in that regard.


As somebody that maintains infrastructure and does a lot of QA 
(ci.debian.net) and is involved in transitions and migrations where I 
see a lot of flaky tests, I'm biased. But really, a test that fails more 
often than say 1/5 times is RC in my book. I didn't check the ratio of 
php-excimer failures.


Paul


OpenPGP_signature
Description: OpenPGP digital signature


Bug#1028041: php-excimer: FTBFS on mipsel

2023-02-02 Thread Kunal Mehta

severity 1028041 normal
thanks

Hi,

On 1/6/23 13:47, Adrian Bunk wrote:

On Fri, Jan 06, 2023 at 08:59:08AM +0100, Paul Gevers wrote:

=
FAILED TEST SUMMARY
-
ExcimerProfiler CPU profile [tests/cpu.phpt]
=

Please fix ASAP to not block the php8.2 transition.


It built after I gave it back to build on the right buildd,
so it's now rebuilt with PHP 8.2.


Thanks.


The issue might depend on buildd speed, or some weird difference
between buildds.


Looking through my email, it's flaked before in the past (#1014801). The 
test in question[1] has the following comment:


// Test aggregateByFunction
// Typically the parent functions foo() and bar() will have self=0 and
// inclusive ~= 30. The other 4 functions will have a count of about 
30/4 = 7.5.
// The probability of C::member() or baz() having a count of zero is 
about 1 in 5600.


Maybe the known flakiness is worse due to something on the mipsel build? 
I'll ask the upstream author. Worst case we can just skip the test on 
mipsel.


I'm lowering the severity because it no longer blocks the 8.2 transition 
(please revert me if I'm wrong on that).


[1] 
https://salsa.debian.org/mediawiki-team/php-excimer/-/blob/master/tests/cpu.phpt


-- Kunal



Bug#1028041: php-excimer: FTBFS on mipsel

2023-01-06 Thread Adrian Bunk
On Fri, Jan 06, 2023 at 08:59:08AM +0100, Paul Gevers wrote:
> Source: php-excimer
> Version: 1.0.4-1
> Severity: serious
> Tags: ftbfs
> Justification: FTBFS
> 
> Dear maintainer,
> 
> The php8.2 transition started recently, and while going over the
> packages involved I noticed that your latest upload (currently in
> unstable only) is failing to build from source on mipsel.
> 
> It fails its tests.
> =
> FAILED TEST SUMMARY
> -
> ExcimerProfiler CPU profile [tests/cpu.phpt]
> =
> 
> Please fix ASAP to not block the php8.2 transition.

It built after I gave it back to build on the right buildd,
so it's now rebuilt with PHP 8.2.

The issue might depend on buildd speed, or some weird difference 
between buildds.

> Paul
>...

cu
Adrian



Bug#1028041: php-excimer: FTBFS on mipsel

2023-01-06 Thread Paul Gevers
Source: php-excimer
Version: 1.0.4-1
Severity: serious
Tags: ftbfs
Justification: FTBFS

Dear maintainer,

The php8.2 transition started recently, and while going over the
packages involved I noticed that your latest upload (currently in
unstable only) is failing to build from source on mipsel.

It fails its tests.
=
FAILED TEST SUMMARY
-
ExcimerProfiler CPU profile [tests/cpu.phpt]
=

Please fix ASAP to not block the php8.2 transition.

Paul

https://buildd.debian.org/status/fetch.php?pkg=php-excimer&arch=mipsel&ver=1.0.4-2&stamp=1672548054&raw=0