1452935 - Remove now-empty `addons.activeExperiment` and check pipeline impact

Status: RESOLVED FIXED

(Toolkit :: Telemetry, enhancement, P1)

Description

[Jan-Erik Rediger [:janerik] (Away until 2024-12-23)]

Since bug 1443608 `addons.activeExperiment` is already empty, bug 1420908 removes most of the machinery.

Before we finally remove the object from the ping payload, we should double-check the pipeline impact.

Comment 1

[Jan-Erik Rediger [:janerik] (Away until 2024-12-23)]

Attachment: Remove empty `addons.activeExperiment`

MozReview-Commit-ID: 2CE6RscQjRV

Comment 2

[Georg Fritzsche [:gfritzsche]]

Comment on attachment 8968566 [details] [diff] [review]
Remove empty `addons.activeExperiment`

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

This looks good, thanks.

Let's also update the environment documentation, in this patch or a separate patch on this bug.
How about we remove `activeExperiment` from the ping format section and add a version history, similar to the main ping documentation?

Comment 3

[Jan-Erik Rediger [:janerik] (Away until 2024-12-23)]

Attachment: Remove obsolete documentation for `addons.activeExperiment`

MozReview-Commit-ID: 3TkQMEWJh6T

Comment 4

[Georg Fritzsche [:gfritzsche]]

Comment on attachment 8970856 [details] [diff] [review]
Remove obsolete documentation for `addons.activeExperiment`

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

That looks good, cheers!

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0b908b54e000
Remove empty `addons.activeExperiment` r=gfritzsche
https://hg.mozilla.org/integration/mozilla-inbound/rev/99a0c6510501
Remove obsolete documentation for `addons.activeExperiment` r=gfritzsche

Comment 6

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/0b908b54e000
https://hg.mozilla.org/mozilla-central/rev/99a0c6510501