1831144 - Land Fox Doodle set to default experiment treatment in MC

Status: RESOLVED FIXED

(Firefox :: Messaging System, enhancement, P2)

Comment 1

[Jason Prickett]

Attachment: Bug 1831144 - Added fox doodle set default spotlight to onboarding messages r?#omc-reviewers

Comment 2

[Phabricator Automation]

Comment on attachment 9332202 [details]
Bug 1831144 - Added fox doodle set default spotlight to onboarding messages r?#omc-reviewers

Revision D177388 was moved to bug 1831145. Setting attachment 9332202 [details] to obsolete.

Comment 3

[Jason Prickett]

[Tracking Requested - why for this release]: We would like to add this message to Fx 114 using localized strings so that we can end the rollout experiment that serves the same message remotely.

Comment 4

[Pascal Chevrel:pascalc]

We are in our last week of beta, this needs to land soon to be uplifted. Also, if there is localization impact please make sure that the l10n-drivers team is aware, thanks!

Comment 5

[Jason Prickett]

Thanks, Pascal. We landed strings in 114 nightly, so no localization impact here.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/071a531f4b98
Added fox doodle set default spotlight to onboarding messages r=omc-reviewers,aminomancer

Comment 7

[Jason Prickett]

We've decided not to uplift this to ensure it gets a full nightly cycle for testing.

Comment 8

[Cosmin Sabou [:CosminS]]

Backed out for causing mochitest plain failures on test_ext_geolocation.html and test_ext_scripting_contentScripts.html.

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&searchStr=plain&revision=071a531f4b98785329fed91898367412000822bf&selectedTaskRun=VysJF6SmSpWfqcfLWFLhjw.0

Failure logs:

• https://treeherder.mozilla.org/logviewer?job_id=417686296&repo=autoland
• https://treeherder.mozilla.org/logviewer?job_id=417687610&repo=autoland

Backout link: https://hg.mozilla.org/integration/autoland/rev/68fc6d938b33f0464a49e5287fa2336d70643666

Comment 9

[Jason Prickett]

Seems this was due to a bad rebase. Rebased on latest mc, and ran the problematic tests locally which are both passing now.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a934ffbfaecf
Added fox doodle set default spotlight to onboarding messages r=omc-reviewers,aminomancer

Comment 11

[Sandor Molnar[:smolnar]]

Backed out for causing mochitest plain failures on test_ext_geolocation.html and test_ext_scripting_contentScripts.html.

Backout link: https://hg.mozilla.org/integration/autoland/rev/ffbe01a5f5e4c81b99d23474f5a4ed48a9e8750f

Push with failures

Failure log -> TEST-UNEXPECTED-FAIL | toolkit/components/extensions/test/mochitest/test_ext_scripting_contentScripts.html | Test timed out. -

Failure log -> TEST-UNEXPECTED-FAIL | toolkit/components/extensions/test/mochitest/test_ext_geolocation.html | Test timed out

Comment 12

[Punam Dahiya [:pdahiya]]

Product ok to land message in tree in Fx116 and should be ok to not uplift in Fx115

Comment 13

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

There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:jprickett, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.

Comment 15

[Jason Prickett]

Attachment: Bug 1831144 - Add fox doodle spotlight message and fix failure of unrelated tests r?#omc-reviewers

Comment 16

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c2f47ac0fbb6
Add fox doodle spotlight message and fix failure of unrelated tests r=omc-reviewers,aminomancer

Comment 17

[Iulian Moraru]

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