Page MenuHomePhabricator

Configure MetricsPlatform to depend on GrowthExperiments in CI
Closed, ResolvedPublic1 Estimated Story Points

Description

Background

In T404058: Add MetricsPlatform to gated extensions, I decided not to make MetricsPlatform a gated extension but rather to adopt the approach suggested in https://phabricator.wikimedia.org/T403560#11158409 and make MetricsPlatform and GrowthExperiments depend on one another. This task covers doing that.

AC

  • MetricsPlatform's tests run when GrowthExperiments' tests run
  • GrowthExperiments' test run when MetricsPlatform's tests run

Event Timeline

phuedx triaged this task as High priority.Sep 26 2025, 10:40 AM
phuedx set the point value for this task to 1.

To confirm, you want to do this instead of T404058, and you're OK for it to go ahead now?

To confirm, you want to do this instead of T404058, and you're OK for it to go ahead now?

Yes and yes.

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

[integration/config@master] Zuul: [mediawiki/extensions/MetricsPlatform] Depend on GrowthExperiments

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

Change #1192895 merged by jenkins-bot:

[integration/config@master] Zuul: [mediawiki/extensions/MetricsPlatform] Depend on GrowthExperiments

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

Mentioned in SAL (#wikimedia-releng) [2025-10-01T15:32:15Z] <James_F> Zuul: [mediawiki/extensions/MetricsPlatform] Depend on GrowthExperiments, for T405715

OK, deployed and rechecked tje latest merged patch, https://gerrit.wikimedia.org/r/c/mediawiki/extensions/MetricsPlatform/+/1183672, which pulls in the GrowthExperiments extension in its Zuul cloner repo pick list (success) and still passes CI (success).

I think this can be declared Resolved? Leaving sign-off of acceptance criteria to @phuedx for confirmation.