Existing cfr experiment results in TypeError "target.features is undefined"
Categories
(Firefox :: Nimbus Desktop Client, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr91 | --- | unaffected |
firefox94 | --- | unaffected |
firefox95 | --- | verified |
firefox96 | --- | verified |
People
(Reporter: Mardak, Assigned: andreio)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details |
I happened to have force enrolled in cfr experiment before bug 1739666 landed, so this could be similar to someone already in a cfr experment in 95 then upgrading to 96?
TypeError: can't access property "find", target.features is undefined
Stack:
get@resource://nimbus/ExperimentAPI.jsm:80:40
_experimentsAPILoader@resource://activity-stream/lib/ASRouter.jsm:347:25
Where it seems to be trying to access…
// { slug: "optin-emotive-pin-firefox-modal", active: true, branch: Proxy }
let experimentData = ExperimentAPI.getExperiment({ featureId: "cfr" });
// { slug: "treatment-b", ratio: 1, feature: { value: {…}, enabled: true, featureId: "cfr" } }
experimentData.branch["cfr"]
So features
doesn't exist on branch
but there is a single feature
Assignee | ||
Comment 1•3 years ago
|
||
Updated•3 years ago
|
Comment 2•3 years ago
|
||
Set release status flags based on info from the regressing bug 1739666
Updated•3 years ago
|
Updated•3 years ago
|
Comment 4•3 years ago
|
||
bugherder |
Updated•3 years ago
|
Assignee | ||
Comment 5•3 years ago
|
||
Comment on attachment 9250807 [details]
WIP: Bug 1741250 - Existing cfr experiment results in TypeError 'target.features is undefined'
Beta/Release Uplift Approval Request
- User impact if declined: This regression break backwards compatibility for experiment recipes when users upgrade from 94 to 95 and could potentially impact any experiment that's live at that time.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: To confirm the fix you should force enroll into the test recipe from Staging
test-tcp-rollout-experimenter-v3
in Firefox 94 then open the same profile in Firefox Nightly. Check in ASRouterAdmin that the recipe spotlight message is present as expected. - List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Small change covered by testing. Only touches Nimbus code not a Firefox feature.
- String changes made/needed:
Assignee | ||
Updated•3 years ago
|
Comment 6•3 years ago
|
||
I have verified that this issue is no longer reproducible with the latest Firefox Nightly (94.0.1 Build ID - 20211103134640), and the latest Firefox Nightly (96.0a1 Build ID - 20211118034937) using the following steps:
- Open the Firefox 94.0.1 Release browser.
- Enroll in the "treatment" branch of the "Test TCP rollout experimenter" rollout.
- Close the browser and open the same profile with the latest Firefox Nightly browser.
- Navigate to the "about:newtab#devtools" page.
- Observe the displayed messages.
After following the steps from above I can confirm that the "AS Router" page is successfully populated and the "test-tcp-rollout-experimenter:treatment" message is displayed.
Comment 7•3 years ago
|
||
Comment on attachment 9250807 [details]
WIP: Bug 1741250 - Existing cfr experiment results in TypeError 'target.features is undefined'
Low risk, approved for 95 beta 9, thanks.
Comment 8•3 years ago
|
||
bugherder uplift |
Comment 9•3 years ago
|
||
I have verified that this issue is no longer reproducible with the latest Firefox Nightly (94.0.1 Build ID - 20211103134640), and the latest Firefox Beta (95.0b9 Build ID - 20211118185700) using the following steps:
Open the Firefox 94.0.1 Release browser.
Enroll in the "treatment" branch of the "Test TCP rollout experimenter" rollout.
Close the browser and open the same profile with the latest Firefox Nightly browser.
Navigate to the "about:newtab#devtools" page.
Observe the displayed messages.
After following the steps from above I can confirm that the "AS Router" page is successfully populated and the "test-tcp-rollout-experimenter:treatment" message is displayed.
Updated•3 years ago
|
Updated•3 years ago
|
Description
•