Growth Experiments
Repository: https://gerrit.wikimedia.org/g/mediawiki/extensions/GrowthExperiments/
File: includes/ExperimentUserDefaultsManager.php
ReaderExperiments
Repository: https://gerrit.wikimedia.org/g/mediawiki/extensions/ReaderExperiments/
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T415546 [EPIC] Improve data collection and experiment ergonomics | |||
| Open | Sgs | T375198 Fully adopt TestKitchen for experiment enrollment | |||
| Open | None | T407803 [Epic] Rename Experimentation Lab/Experiment Platform to... Test Kitchen | |||
| Resolved | cjming | T407905 Update name in GrowthExperiments and ReaderExperiments extensions |
Change #1218398 had a related patch set uploaded (by Clare Ming; author: Clare Ming):
[mediawiki/extensions/GrowthExperiments@master] Update Metrics Platform references to Test Kitchen
Change #1219236 had a related patch set uploaded (by Clare Ming; author: Clare Ming):
[mediawiki/extensions/WikiLambda@master] Update references to Test Kitchen
Change #1219237 had a related patch set uploaded (by Clare Ming; author: Clare Ming):
[mediawiki/extensions/ReaderExperiments@master] Update `Metrics Platform` references to `Test Kitchen`
I have updated the ticket description to remove the references to Wikilambda. As we have discussed, there is actually no work related to the renaming itself in that extension. The instrument created there was implemented before MetricsPlatform existed. In reality they are only using EventLogging::getMetricsPlatformClient().submitInteraction() (PHP SDK) as a way to send events, which is related to T409913: Create PHP equivalent of mw.testKitchen.getInstrument()
Change #1219236 abandoned by Clare Ming:
[mediawiki/extensions/WikiLambda@master] Update references to Test Kitchen
Reason:
N/A
Change #1219237 merged by jenkins-bot:
[mediawiki/extensions/ReaderExperiments@master] Update `Metrics Platform` references to `Test Kitchen`
Change #1218398 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] Update Metrics Platform references to Test Kitchen
@cjming GrowthExperiment extension is also using EventLogging.MetricsClientFactory service which is something we would like to deprecate and remove as part of T408059: [GOAL] Tidy up EventLogging. I have just created a ticket to deprecate and remove that service at some point (T416020: Deprecate and remove EventLogging.MetricsClientFactory service).
I'm also filing tickets to migrate all the instruments that are using it and I'm here just to confirm with you that (1) this ticket is only to rename the extension itself (from MetricsPlatform to TestKitchen) and we should file a separate ticket to address the EventLogging.MetricsClientFactory service also for GrowthExperiment extension instead of (2) using this ticket also for that purpose. (I like 1)
GrowthExperiments does not use EventLogging.MetricsClientFactory since rEGRE5f687c8d06304b37c25fa14adaf506a922646e59 so I'm tentatively resolving. @Sfaci please re-open or file a task if you think something is missing to be migrated in GE.
Change #1247698 had a related patch set uploaded (by Jforrester; author: Jforrester):
[integration/config@master] Zuul: [mediawiki/extensions/GrowthExperiments] Drop MetricsPlatform dep
Change #1247698 merged by jenkins-bot:
[integration/config@master] Zuul: [mediawiki/extensions/GrowthExperiments] Drop MetricsPlatform dep
Change #1293735 had a related patch set uploaded (by Hashar; author: Hashar):
[integration/quibble@master] utils: fullrun-extensions: s/MetricsPlatform/TestKitchen/
Change #1293735 merged by jenkins-bot:
[integration/quibble@master] utils: fullrun-extensions: s/MetricsPlatform/TestKitchen/