Page MenuHomePhabricator

Flaky PHPUnit failures on PHP 7.1 for BlueSpiceNamespaceManager
Closed, ResolvedPublic

Description

https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/BlueSpiceNamespaceManager/+/497119/ fails PHPUnit tests on PHP 7.1.

I am not a regular contributor on BlueSpice so I'm filing this task hoping that you could take a look.

Thanks.

Event Timeline

xSavitar renamed this task from PHPUnit failures on PHP 7.1 for BlueSpiceNamespaceManager to Flaky PHPUnit failures on PHP 7.1 for BlueSpiceNamespaceManager.Mar 18 2019, 12:34 PM

This is a bit difficult to investigate because jenkins logs have been garbage collected and they weren't copied here. Maybe someone could submit a DNM commit in BlueSpiceNamespaceManager and trigger PHP jobs until they fail? Or test it locally with PHP 7.1 if it's reproducible?

This may has to do with php op-cache. As we recently found out, that the files in BlueSpiceFoundation/config/* - which are edited by the BlueSpiceNamespaceManager in the tests - are not invalidated fast enough in op cache to be up to date when read in again.
However i dont now if op-cache is running within the ci tests

Change 552020 had a related patch set uploaded (by Pwirth; owner: Pwirth):
[mediawiki/extensions/BlueSpiceNamespaceManager@master] Deactivate broken test.

https://gerrit.wikimedia.org/r/552020

Change 555026 had a related patch set uploaded (by ItSpiderman; owner: Pwirth):
[mediawiki/extensions/BlueSpiceNamespaceManager@REL1_31_dev] Deactivate broken test.

https://gerrit.wikimedia.org/r/555026

Change 555027 had a related patch set uploaded (by ItSpiderman; owner: Pwirth):
[mediawiki/extensions/BlueSpiceNamespaceManager@REL1_31] Deactivate broken test.

https://gerrit.wikimedia.org/r/555027

Change 555027 merged by jenkins-bot:
[mediawiki/extensions/BlueSpiceNamespaceManager@REL1_31] Deactivate broken test.

https://gerrit.wikimedia.org/r/555027

Change 555026 merged by Pwirth:
[mediawiki/extensions/BlueSpiceNamespaceManager@REL1_31_dev] Deactivate broken test.

https://gerrit.wikimedia.org/r/555026

Change 552020 merged by jenkins-bot:
[mediawiki/extensions/BlueSpiceNamespaceManager@master] Deactivate broken test.

https://gerrit.wikimedia.org/r/552020

Change 657307 had a related patch set uploaded (by Pwirth; owner: Pwirth):
[mediawiki/extensions/BlueSpiceNamespaceManager@REL1_35] Deactivate broken test.

https://gerrit.wikimedia.org/r/657307

Change 657307 merged by jenkins-bot:
[mediawiki/extensions/BlueSpiceNamespaceManager@REL1_35] Deactivate broken test.

https://gerrit.wikimedia.org/r/657307

Pwirth claimed this task.

tests have been marked as broken.