Page MenuHomePhabricator

Quibble CI fails with MediaWiki\Skins\Vector\Tests\Structure\PerformanceBudgetTest::testTotalModulesSize with data set "vector-2022"
Closed, ResolvedPublic

Description

As seen on https://integration.wikimedia.org/ci/job/integration-quibble-fullrun-extensions-phpunit/510/console

13:42:08 1) MediaWiki\Skins\Vector\Tests\Structure\PerformanceBudgetTest::testTotalModulesSize with data set "vector-2022" ('vector-2022', array('50KB', '21KB'))
13:42:08 T346813: Performance budget for styles in skin vector-2022 on main article namespace has been exceeded. Total size of styles modules is 22.1 Kbs is greater than the current budget size of 22 Kbs (see Vector/bundlesize.config.json).
13:42:08 If you are adding code on page load, please reduce styles that you are loading on page load or talk to the web team about increasing the budget.
13:42:08 If you are not adding code, and this seems to be an error, it is possible that something running without CI has bypassed this check and we can address this separately.Please reach out to the web team to discuss this via Phabricator or #talk-to-web.[PLEASE DO NOT SKIP THIS TEST. If this is blocking a deploy this might signal a potential performance regression with the desktop site.Instead of skipping the test you can increase the value in bundlesize.config.json and create a ticket to investigate this error. If the error is > 1kb please tag this as a train blocker.Please tag the ticket #Web-Team-Backlog.

T346813: Vector performance budget's should consider modules added by hooks

Can be tested with https://gerrit.wikimedia.org/r/c/integration/quibble/+/1043723

Event Timeline

kostajh raised the priority of this task from High to Unbreak Now!.
kostajh updated the task description. (Show Details)
kostajh added a subscriber: hashar.

This is blocking merges to Quibble, so marking as UBN.

One thing to check, is that the list of extension dependencies https://gerrit.wikimedia.org/r/c/integration/quibble/+/1035364/28/utils/ci-fullrun-extensions.sh is up to date with what we are using in CI nowadays. Presumably it differs, which is why we see the failure in Quibble's CI but not in MediaWiki's CI.

Sorry we will get this fixed today. We thlught we had fixed this yesterday.

Jdlrobson claimed this task.