Description
Now that T408186: Configure experiments with stream, schema and contextual attributes has been shipped and we have configured already all the existing experiments with their stream_name and contextual_attributes in Test Kitchen UI, Experiment#setStream(streamName) is no longer needed and can be deprecated/removed. And that also means that wgTestKitchenExperimentStreamNames is not needed either.
Because the existing experiments have already their own configuration, the ExperimentManager (for both JS and PHP SDKs) will be instantiating experiments with all the needed configurations, so the following would be a valid approach:
- Confirm that all existing experiment are updated with their own stream_name and contextual_attributes in Test Kitchen UI/API (/api/v1/experiments endpoint)
- Remove Experiment#setStream call from the experiments code where it's being used
- Remove Experiment#setStream from both Test Kitchen JS and PHP SDKs
Acceptance Criteria
- We have checked that all existing experiment has their own configuration in Test Kitchen UI (stream_name + contextual_attributes)
- Some previous work was already done. Some details here
- Experiment#setStream() has been deprecated for both JS and PHP SDKs
- Usages of Experiment::setStream() has been removed
- Experiment#setStream() has been removed from both JS and PHP SDKs (merged)
- wgTestKitchenExperimentStreamNames has been removed from mediawiki-config (ready to be deployed)
- Documentation has been updated
- https://wikitech.wikimedia.org/wiki/Test_Kitchen/Conduct_an_experiment#Using_custom_schema_and/or_stream
- https://wikitech.wikimedia.org/wiki/Test_Kitchen/Measure_product_health#Using_custom_schema_and/or_stream
- https://wikitech.wikimedia.org/wiki/Test_Kitchen/SDK/JavaScript_SDK
- https://wikitech.wikimedia.org/wiki/Test_Kitchen/SDK/PHP_SDK