Closed Bug 1474413 Opened 6 years ago Closed 6 years ago

Update add-on studies in place when the recipe changes.

Categories

(Firefox :: Normandy Client, enhancement, P1)

Product:
Firefox
Component:
Normandy Client
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 67
Tracking Flags:
Tracking Status
firefox-esr52 --- unaffected
firefox-esr60 --- wontfix
firefox61 --- wontfix
firefox62 --- wontfix
firefox63 --- wontfix
firefox65 --- wontfix
firefox66 --- wontfix
firefox67 --- fixed

People

(Reporter: mythmon, Assigned: rehandalal+mozilla)

References

Details

Attachments

(1 file, 1 obsolete file)

Bug 1474413 - Update addon studies in place r=mythmon
(deleted), text/x-phabricator-request
Details
When the add-on listed in an add-on (aka opt-out) study does not match the currently installed add-on for the study, the old add-on should be removed and replaced with the new add-on. Updating add-ons in place could cause strange effects in relation to the shield studies. We'll have to take care to not make existing studies think that the study is ending.
If my understanding on this matter is correct, we want this change in order not to lose and reuse the user study bucket when the study add-on needs adjustments or to be completely changed. I'm not sure at this point what changes are required on Normandy client side and risks involved plus I assume we'll want the changes uplifted as well? I would suggest an additional telemetry status for this change. Setting flags accordingly; Mike can you please update the tracking flags in order to reflect what versions we will target with this change?
status-firefox61: --- → affected
status-firefox62: --- → affected
status-firefox63: --- → affected
status-firefox-esr52: --- → unaffected
status-firefox-esr60: --- → affected
Flags: qe-verify+
Flags: needinfo?(mcooper)
This change won't be small, and I plan to do it as a part of a migration to a local action. Because of this, I'm not planning to request uplift for the feature.
status-firefox61: affected → wontfix
status-firefox62: affected → wontfix
status-firefox-esr60: affected → wontfix
Flags: needinfo?(mcooper)
Depends on: 1440780
Priority: -- → P2
Assignee: nobody → rdalal
Priority: P2 → P1
Depends on: 1528458
Attached file Bug 1474413 - Update addon studies in place r?mythmon (obsolete) (deleted) —
Attachment #9044849 - Attachment is obsolete: true
Keywords: checkin-needed

Failed to land:
Details: We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again. (255, 'applying /tmp/tmpVd165v\npatching file toolkit/components/normandy/test/browser/head.js\nHunk #1 succeeded at 11 with fuzz 2 (offset 2 lines).\npatching file toolkit/components/normandy/test/browser/browser_actions_AddonStudyAction.js\nHunk #4 FAILED at 166\nHunk #12 FAILED at 882\n2 out of 12 hunks FAILED -- saving rejects to file toolkit/components/normandy/test/browser/browser_actions_AddonStudyAction.js.rej\npatching file toolkit/components/normandy/lib/TelemetryEvents.jsm\nHunk #1 FAILED at 26\n1 out of 1 hunks FAILED -- saving rejects to file toolkit/components/normandy/lib/TelemetryEvents.jsm.rej\nabort: patch failed to apply', '')

Please take a look.

Flags: needinfo?(rdalal)
Attachment #9043067 - Attachment description: Bug 1474413 - Update addon studies in place r?mythmon → Bug 1474413 - Update addon studies in place r=mythmon
Keywords: checkin-needed

Pushed by nbeleuzu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a920a704ff6b
Update addon studies in place r=mythmon,glasserc

Keywords: checkin-needed

https://hg.mozilla.org/mozilla-central/rev/a920a704ff6b

Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox67: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
Whiteboard: [qa-triaged]
QA Whiteboard: [qa-triaged]
Whiteboard: [qa-triaged]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: