1376599 - Add "type" key to experiment annotations

Status: RESOLVED FIXED

(Toolkit :: Telemetry, enhancement, P1)

Description

[bugzilla]

Currently, if pref rollouts will end up using experiment annotations, this will blow up the lua filter that creates the telemetry-experiments source. If we add a "type" field to the annotation (and get normandy to annotate those with its own type) we should be able to ignore pref rollouts in the filter. We *might not* have enough time to do this before the rollouts go out, in which case we need to work with normandy to use a naming convention that we'll add to the filter (but we should still add the type annotation.)

Comment 1

[Georg Fritzsche [:gfritzsche]]

Attachment: Allow annotating experiments with a type

Comment 2

[Alessio Placitelli [:Dexter]]

Comment on attachment 8882042 [details] [diff] [review]
Allow annotating experiments with a type

Review of attachment 8882042 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good!

Comment 3

[Georg Fritzsche [:gfritzsche]]

Comment on attachment 8882042 [details] [diff] [review]
Allow annotating experiments with a type

Approval Request Comment
[Feature/Bug causing the regression]: Telemetry experiments annotations.
[User impact if declined]: Data pipeline can't effectively filter out pref-flip experiments (as we can't tell them apart from pref rollouts etc.).
[Is this code covered by automated tests?]: Yes.
[Has the fix been verified in Nightly?]: Pending.
[Needs manual test from QE? If yes, steps to reproduce]: No.
[List of other uplifts needed for the feature/fix]: None.
[Is the change risky?]: No.
[Why is the change risky/not risky?]: Its a small optional addition to the Telemetry annotation API.
[String changes made/needed]: None.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/72aa740f3363
Allow annotating experiments with a type. r=Dexter

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3dac05357e71
Allow annotating experiments with a type: Fix eslint issues. r=eslint-fix on a CLOSED TREE

Comment 6

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/72aa740f3363
https://hg.mozilla.org/mozilla-central/rev/3dac05357e71

Comment 7

[Julien Cristau [:jcristau]]

Comment on attachment 8882042 [details] [diff] [review]
Allow annotating experiments with a type

add type annotation for experiments, beta55+

you may want to s/lenght/length/ in experiments.rst.

Comment 8

[Alessio Placitelli [:Dexter]]

(In reply to Julien Cristau [:jcristau] from comment #7)
> Comment on attachment 8882042 [details] [diff] [review]
> Allow annotating experiments with a type
> 
> add type annotation for experiments, beta55+
> 
> you may want to s/lenght/length/ in experiments.rst.

Documentation is built from mozilla-central, so that typo should be fixed there. We should still be good to uplift this patch to beta. I've filed bug 1378345 for the typo.

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-beta/rev/e2dc26e7f164
https://hg.mozilla.org/releases/mozilla-beta/rev/95bd58aaf678

Comment 10

[Andrei Vaida [:avaida]]

(In reply to Georg Fritzsche [:gfritzsche] from comment #3)
> [Is this code covered by automated tests?]: Yes.
> [Has the fix been verified in Nightly?]: Pending.
> [Needs manual test from QE? If yes, steps to reproduce]: No.

Setting qe-verify- based on Georg's assessment on manual testing needs and the fact that this fix has automated coverage.