Closed Bug 1569330 Opened 3 months ago Closed 3 months ago

Decouple Normandy recipe runner from Telemetry

Categories

(Firefox :: Normandy Client, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
Firefox 70
Tracking Status
firefox70 --- fixed

People

(Reporter: rehan, Assigned: rehan)

References

Details

Attachments

(1 file)

In the checkPrefs method in RecipeRunner.jsm we currently check whether Telemetry is enabled or not and accordingly decide whether the recipe runner should be run at all. As part of the move to using Normandy for hotfixes we want to loosen this coupling.

The existing study actions will already respect the Shield enabled pref which is automatically disabled when users opt out of Telemetry. So no changes should be required there.

Upon further investigation it seems that this is not true:

The existing study actions will already respect the Shield enabled pref which is automatically disabled when users opt out of Telemetry. So no changes should be required there.

We will need to modify the existing actions or change the layout of about:preferences.

Blocks: 1473874
Keywords: checkin-needed

Pushed by apavel@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4e14ea350022
Disable telemetry check in Normandy recipe runner r=mythmon,nalexander

Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
You need to log in before you can comment on or make changes to this bug.