Hi Johannes,

Just a simple observation - you don’t need to restart FPM as you don’t do any 
modifications to the PHP FPM configuration.

So just drop the line:

systemctl restart php7.2-fpm

from d/t/general and the package tests will be fine again.

Ondrej
--
Ondřej Surý
ond...@sury.org



> On 25 Oct 2018, at 16:41, Paul Gevers <elb...@debian.org> wrote:
> 
> Source: rss-bridge
> Version: 2018-07-17-1
> User: debian...@lists.debian.org
> Usertags: needs-update
> Control: affects -1 src:php-defaults
> 
> [X-Debbugs-CC: debian...@lists.debian.org, php-defau...@packages.debian.org]
> 
> Dear maintainers,
> 
> With a recent upload of php-defaults the autopkgtest of rss-bridge fails
> in testing when that autopkgtest is run with the binary packages of
> php-defaults from unstable. It passes when run with only packages from
> testing. In tabular form:
>                       pass            fail
> php-defaults           from testing    66
> rss-bridge             from testing    2018-07-17-1
> versioned deps [0]     from testing    from unstable
> all others             from testing    from testing
> 
> I copied some of the output at the bottom of this report. If I
> understand the error correctly, you restart php7.2 while your test
> doesn't depend on php7.2. I think you need to update your test be smart
> enough for the case where the default php version is updated.
> 
> Currently this regression is contributing to the delay of the migration
> of php-defaults to testing [1]. Of course, php-defaults shouldn't just
> break your autopkgtest (or even worse, your package), but it seems to me
> that the change in php-defaults was intended and your package needs to
> update to the new situation. If needed, please change the bug's severity.
> 
> If this is a real problem in your package (and not only in your
> autopkgtest), the right binary package(s) from php-defaults should
> really add a versioned Breaks on the unfixed version of (one of your)
> package(s). Note: the Breaks is nice even if the issue is only in the
> autopkgtest as it helps the migration software to figure out the right
> versions to combine in the tests.
> 
> More information about this bug and the reason for filing it can be found on
> https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation
> 
> Paul
> 
> [1] https://qa.debian.org/excuses.php?package=php-defaults
> 
> https://ci.debian.net/data/autopkgtest/testing/amd64/r/rss-bridge/1204134/log.gz
> 
> autopkgtest [04:42:26]: test general: [-----------------------
> + rm /etc/nginx/sites-enabled/default
> + cp /usr/share/doc/rss-bridge/examples/nginx.conf
> /etc/nginx/sites-available/rss-bridge
> + ln -s ../sites-available/rss-bridge /etc/nginx/sites-enabled/rss-bridge
> + systemctl restart nginx
> + systemctl restart php7.2-fpm
> Failed to restart php7.2-fpm.service: Unit php7.2-fpm.service not found.
> autopkgtest [04:42:27]: test general: -----------------------]
> 

Reply via email to