CheckUser REL1_44 CI blocked with 'Error: Class "Pimple\Container" not found'
Closed, DuplicatePublic
Actions

Assigned To

None

Authored By

	Dreamy_Jazz
	May 8 2025, 7:28 AM

Description

The CheckUser extension CI for MediaWiki release 1.44 is blocked with the following error:

22:58:03 [4d6ad0af7dcdf758bdf31d08] [no req]   Error: Class "Pimple\Container" not found
22:58:03 Backtrace:
22:58:03 from /workspace/src/extensions/Flow/includes/Container.php(5)
22:58:03 #0 /workspace/src/includes/AutoLoader.php(226): require_once()
22:58:03 #1 /workspace/src/extensions/Flow/maintenance/FlowSetUserIp.php(41): AutoLoader::autoload()
22:58:03 #2 /workspace/src/maintenance/includes/LoggedUpdateMaintenance.php(52): Flow\Maintenance\FlowSetUserIp->doDBUpdates()
22:58:03 #3 /workspace/src/maintenance/update.php(210): MediaWiki\Maintenance\LoggedUpdateMaintenance->execute()
22:58:03 #4 /workspace/src/maintenance/includes/MaintenanceRunner.php(694): UpdateMediaWiki->execute()
22:58:03 #5 /workspace/src/maintenance/doMaintenance.php(100): MediaWiki\Maintenance\MaintenanceRunner->run()
22:58:03 #6 /workspace/src/maintenance/update.php(311): require_once(string)
22:58:03 #7 {main}
22:58:03 <<< Finish: Install MediaWiki, db=<MySQL /workspace/db/quibble-mysql-5_tnm0fp/socket>, in 3.313 s

Details

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	Zuul: Drop StructuredDiscussions as a CI dependency on release branches	integration/config	master	+9 -0
	Zuul: Expand WebAuthn CI disable to all non-master/non-wmf branches	integration/config	master	+4 -3

Customize query in gerrit

Related Objects

Mentioned In: T404868: composer merge plugin does not fail when dependency is not satisfied (was: Error: Class "Pimple\Container" not found)
Mentioned Here: T363639: web-auth/webauthn-lib must be upgraded to 4+ for PHP 8.2+ support
rCICFa4a03c44ef1c: Zuul: [mediawiki/extensions/CheckUser] Add GrowthExperiments dependency
T386435: UserInfoCard: Create service to return user info card data for an account

Event Timeline

Dreamy_Jazz created this task.May 8 2025, 7:28 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 8 2025, 7:28 AM

Dreamy_Jazz added a project: StructuredDiscussions.May 8 2025, 9:17 AM

Restricted Application added a project: Growth-Team. · View Herald TranscriptMay 8 2025, 9:17 AM

Dreamy_Jazz added a project: CheckUser.May 8 2025, 9:17 AM

Restricted Application added a project: Trust and Safety Product Team. · View Herald TranscriptMay 8 2025, 9:17 AM

Failure appears to be with StructuredDiscussions instead of CheckUser.

Blocking the merging of security patch backports https://gerrit.wikimedia.org/r/c/mediawiki/extensions/IPInfo/+/1143191 and https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CheckUser/+/1143192

Hmm. Pimple\Container isn't in the vendor.git for MW for REL1_44; I think this is a CI config issue, that CU is pulling in GrowthExperiments which is pulling in StructuredDiscussions for master, but the config also applies in release branches.

The addition of GrowthExperiments to CheckUser is new as of six weeks ago in a4a03c44ef for T386435. @kostajh / @STran, can you confirm it's OK for that CI to not be tested in REL1_44 (and other non-production release branches)?

In T393670#10804018, @Jdforrester-WMF wrote:

Hmm. Pimple\Container isn't in the vendor.git for MW for REL1_44; I think this is a CI config issue, that CU is pulling in GrowthExperiments which is pulling in StructuredDiscussions for master, but the config also applies in release branches.

The addition of GrowthExperiments to CheckUser is new as of six weeks ago in a4a03c44ef for T386435. @kostajh / @STran, can you confirm it's OK for that CI to not be tested in REL1_44 (and other non-production release branches)?

Is your proposal to revert a4a03c44ef for REL1_44? That would be fine with me.

Change #1143542 had a related patch set uploaded (by Jforrester; author: Jforrester):

[integration/config@master] Zuul: Drop StructuredDiscussions as a CI dependency on release branches

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

gerritbot added a project: Patch-For-Review.May 8 2025, 12:06 PM

In T393670#10804025, @kostajh wrote:

In T393670#10804018, @Jdforrester-WMF wrote:

The addition of GrowthExperiments to CheckUser is new as of six weeks ago in a4a03c44ef for T386435. @kostajh / @STran, can you confirm it's OK for that CI to not be tested in REL1_44 (and other non-production release branches)?

Is your proposal to revert a4a03c44ef for REL1_44? That would be fine with me.

Slightly more hackily, sadly, but yes, fundamentally. OK, will do that.

Change #1143550 had a related patch set uploaded (by Jforrester; author: Jforrester):

[integration/config@master] Zuul: Expand WebAuthn CI disable to all non-master/non-wmf branches

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

Change #1143550 merged by jenkins-bot:

[integration/config@master] Zuul: Expand WebAuthn CI disable to all non-master/non-wmf branches

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

Folding into T363639.

Jdforrester-WMF closed this task as a duplicate of T363639: web-auth/webauthn-lib must be upgraded to 4+ for PHP 8.2+ support.May 8 2025, 1:59 PM

Change #1143542 abandoned by Jforrester: