Add telemetry for control group in adapter experiments #7817

luabud · 2019-10-08T15:56:56Z

Currently we're getting telemetry only for those in the experiment groups for wheels and the new adapter, and not the ones in control

kimadeline · 2019-10-10T15:20:35Z

Prescribed solution:

call sendTelemetryIfInExperiment for the DA control group in src/client/debugger/extension/adapter/activator.ts
call sendTelemetryIfInExperiment for the wheel control group in src/client/debugger/extension/adapter/factory.ts
add telemetry unit tests for both experiments

Note: Experiment telemetry refactoring will be done in #7869

kimadeline · 2019-10-18T23:22:17Z

@luabud @karthiknadig should we close this issue now that the DA experiment is disabled?

luabud · 2019-10-21T18:17:11Z

I'd say we can close this once the experiment is back on and we start getting the telemetry events :)

karthiknadig · 2019-10-21T18:19:14Z

This should be back after we merge #8119

luabud · 2019-10-21T19:03:09Z

Then it's being tracked elsewhere, so closing it.

luabud added bug Issue identified by VS Code Team member as probable bug needs PR important Issue identified as high-priority labels Oct 8, 2019

karthiknadig added this to the 2019 October Sprint 2 milestone Oct 9, 2019

kimadeline mentioned this issue Oct 9, 2019

Separate telemetry from experiment check #7869

Closed

kimadeline self-assigned this Oct 10, 2019

kimadeline mentioned this issue Oct 11, 2019

Add telemetry for control groups of the debug adapter experiments #7938

Merged

10 tasks

kimadeline assigned luabud Oct 15, 2019

luabud closed this as completed Oct 21, 2019

ghost removed the needs PR label Oct 21, 2019

lock bot locked as resolved and limited conversation to collaborators Oct 28, 2019