Maniphest T200762

Integration broken on MassMessage
Closed, ResolvedPublicPRODUCTION ERROR
Actions

Description

https://gerrit.wikimedia.org/r/c/mediawiki/extensions/MassMessage/+/449395 (i18n update) fails on submit.

See https://integration.wikimedia.org/ci/job/quibble-vendor-mysql-hhvm-docker/9212/console :

1) MediaWiki\MassMessage\ApiEditMassMessageListTest::testAdd
BadMethodCallException: Call to a member function getCanonicalServer() on a non-object (null)

extensions/MassMessage/includes/DatabaseLookup.php:29
includes/libs/objectcache/WANObjectCache.php:1242
includes/libs/objectcache/WANObjectCache.php:1115
extensions/MassMessage/includes/DatabaseLookup.php:36
extensions/MassMessage/includes/content/MassMessageListContent.php:324
extensions/MassMessage/includes/content/MassMessageListContent.php:185
includes/content/AbstractContent.php:516
includes/Storage/DerivedPageDataUpdater.php:1240
includes/Storage/DerivedPageDataUpdater.php:1263
includes/Storage/PageUpdater.php:697
includes/page/WikiPage.php:1855
extensions/MassMessage/tests/phpunit/api/ApiEditMassMessageListTest.php:24
tests/phpunit/MediaWikiTestCase.php:469
maintenance/doMaintenance.php:94

All tests contain the same error message: BadMethodCallException: Call to a member function getCanonicalServer() on a non-object (null).

Details

	Project	Branch	Lines +/-	Subject
	mediawiki/extensions/MassMessage	REL1_31	+15 -9	Setup wgCanonicalServer in MassMessageApiTestCase to pass tests
	mediawiki/extensions/MassMessage	master	+15 -9	Setup wgCanonicalServer in MassMessageApiTestCase to pass tests

Customize query in gerrit

Related Objects

Mentioned Here: T142229: PHP Fatal error: Call to a member function getCanonicalServer() on a non-object

Event Timeline

matej_suchanek created this task.Jul 31 2018, 8:12 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 31 2018, 8:12 AM

matej_suchanek triaged this task as Unbreak Now! priority.Jul 31 2018, 8:13 AM

Restricted Application added subscribers: Liuxinyu970226, TerraCodes. · View Herald TranscriptJul 31 2018, 8:13 AM

I think there's better tag, but I cannot find it now

hashar updated the task description. (Show Details)Aug 1 2018, 8:04 AM

Based on the stack trace, could it be an issue with MediaWiki core and the ongoing work on multi content revisions?

It should be preproducible locally via tests/phpunit/phpunit.php --testsuite extensions --group Database. Note CI also injects mediawiki/extensions/LiquidThreads as a dependency.

Umherirrender added a project: Wikimedia-production-error (ARCHIVED -- Shared Build Failure).Aug 11 2018, 9:15 AM

Nikerabbit added a subscriber: Nikerabbit.Aug 13 2018, 6:06 AM

MarcoAurelio added a subscriber: MarcoAurelio.Aug 15 2018, 9:14 PM

Change 453424 had a related patch set uploaded (by Umherirrender; owner: Umherirrender):
[mediawiki/extensions/MassMessage@master] Setup wgCanonicalServer in MassMessageApiTestCase to pass tests

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

gerritbot added a project: Patch-For-Review.Aug 17 2018, 3:51 PM

The failure is on WikiMap::getWiki( $dbname )->getCanonicalServer(); where the dbname comes from $wgConf->getLocalDatabases()
MassMessageApiTestCase sets that global in setUpBeforeClass

Was fixed somedays ago in T142229 for the normal tests. I have copy over the canonical servers to api, hopefully that fixes the test

Jayprakash12345 added a subscriber: Jayprakash12345.Aug 17 2018, 4:00 PM

This task is from 31. jul - the last patch merged on MassMessage is a version update from some days before - https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/MassMessage/+/448171/

When compare the jenkins output, it seems the last patch runs only PHPUnit extensions testsuite (1457 tests) while now it runs PHPUnit extensions suite without Database group (1546 tests) + PHPUnit extensions suite Database group (Tests: 43), so it is running more than before.

A test config change does the different here. No idea what it is, but I have uploaded a fix, which now passed the tests.

Change 453424 merged by jenkins-bot:
[mediawiki/extensions/MassMessage@master] Setup wgCanonicalServer in MassMessageApiTestCase to pass tests

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

Umherirrender closed this task as Resolved.Aug 17 2018, 5:53 PM

Umherirrender claimed this task.

ReleaseTaggerBot added a project: MW-1.32-notes (WMF-deploy-2018-08-21 (1.32.0-wmf.18)).Aug 17 2018, 6:00 PM

Liuxinyu970226 removed a subscriber: Liuxinyu970226.Aug 23 2018, 2:28 PM

CCicalese_WMF moved this task from Inbox to Done on the Multi-Content-Revisions board.Sep 18 2018, 10:09 PM

Ladsgroup removed a project: Patch-For-Review.May 21 2019, 9:28 PM

mmodell changed the subtype of this task from "Task" to "Production Error".Aug 28 2019, 11:08 PM

Change 653156 had a related patch set uploaded (by Umherirrender; owner: Umherirrender):
[mediawiki/extensions/MassMessage@REL1_31] Setup wgCanonicalServer in MassMessageApiTestCase to pass tests

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

gerritbot added a project: Patch-For-Review.Jan 2 2021, 9:39 AM

Change 653156 merged by Umherirrender:
[mediawiki/extensions/MassMessage@REL1_31] Setup wgCanonicalServer in MassMessageApiTestCase to pass tests