1766430 - [Experiment] A validation error is displayed in the "Browser Console" when trying to enroll in the "Covered Regular ECO2204" experiment

Status: VERIFIED FIXED

(Firefox :: Messaging System, defect, P1)

Description

[Marius Coman [:mcoman], Ecosystem QA]

Attachment: ss of the issue.png

[Affected versions]:

• Firefox Beta 100.0 - Build ID: 20220425210429

[Affected Platforms]:

• Windows 10 x64
• macOS 11.6.2

[Prerequisites]:

• Have a Firefox en-* locale build installed.
• Have a Firefox profile older than 28 days
• Firefox is NOT pinned to the taskbar/dock.
• Have the "user.js" file saved to your PC.

[Steps to reproduce]:

1. Open the browser using the profile from the prerequisites.
2. Navigate to the profile folder and paste the "user.js" file from the prerequisites.
3. Restart the browser and focus the "Browser Console".
4. Observe the displayed errors.

[Expected result]:

• No error is displayed and you are successfully enrolled in the experiment.

[Actual result]:

• A validation error is displayed and you are NOT enrolled in the experiment.

[Notes]:

• This issue is not reproducible if a force enrollment is performed instead of a natural one.
• Attached a screenshot of the error.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1752665

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

:barret, since you are the author of the regressor, bug 1752665, could you take a look?
For more information, please visit auto_nag documentation.

Comment 3

[Barret Rennie [:barret] (they/them)]

Attachment: Bug 1766430 - Allow additional properties in Spotlight content r?mardak

Comment 4

[Ed Lee :Mardak]

[Tracking Requested - why for this release]: We need this fix to be able to launch some messaging experiments during 100

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/976c7f10f616
Allow additional properties in Spotlight content r=Mardak

Comment 6

[Barret Rennie [:barret] (they/them)]

Comment on attachment 9273868 [details]
Bug 1766430 - Allow additional properties in Spotlight content r?mardak

Beta/Release Uplift Approval Request

• User impact if declined: This fixes an issue with schema validation that will prevent us from running experiments against Firefox release until 101.
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This is not risky because it is just allowing extra data to appear in the JSON blob for experiments. The alternative is to wait until Fx 101 to run experiments, which would be a high price to pay.
• String changes made/needed:
• Is Android affected?: No

Comment 7

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/976c7f10f616

Comment 8

[Dianna Smith [:diannaS]]

Comment on attachment 9273868 [details]
Bug 1766430 - Allow additional properties in Spotlight content r?mardak

Approved for 100.0rc2

Comment 9

[Dianna Smith [:diannaS]]

https://hg.mozilla.org/releases/mozilla-release/rev/7698c74db11b

Comment 10

[Simona Rosu [:srosu], Ecosystem QA]

This issue was part of the work done on QA-1465 for which we sent a Sign off. Considering this I am marking it as Verified.