Closed
Bug 1425974
Opened 7 years ago
Closed 7 years ago
browser_aboutSearchReset.js is going to permafail when the Gecko version number is bumped to 60
Categories
(Firefox :: Search, enhancement, P1)
Firefox
Search
Tracking
()
VERIFIED
FIXED
Firefox 59
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox57 | --- | unaffected |
firefox58 | --- | unaffected |
firefox59 | + | verified |
People
(Reporter: RyanVM, Assigned: florian)
References
Details
(Whiteboard: [fxsearch])
Attachments
(1 file)
[Tracking Requested - why for this release]: Permafailing tests when the Gecko version number is bumped to 60.
Should we extend the SEARCH_RESET_RESULT probe expiration again, Florian?
Flags: needinfo?(florian)
Updated•7 years ago
|
Priority: -- → P1
Whiteboard: [fxsearch]
Comment 1•7 years ago
|
||
The early/first merge will be Jan. 11th.
Tracking so we can make sure to check back in early January.
Does extending a probe that was scheduled to expire need a data review?
Updated•7 years ago
|
Assignee | ||
Comment 2•7 years ago
|
||
Given that we just shipped this UI in Firefox 57.0.1 (in bug 1419941) and haven't analyzed the results yet, now is the worst possible time to drop this probe. Let's bump it by a couple more releases. I think this is straightforward enough that it could land without data review, flagging Panos to check if he agrees or wants to figure out who to loop in (as bsmedberg handled the review of the previous bumps but is no longer around).
Attachment #8938374 -
Flags: review?(past)
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → florian
Status: NEW → ASSIGNED
Comment 3•7 years ago
|
||
Comment on attachment 8938374 [details] [diff] [review]
bump to 65
Review of attachment 8938374 [details] [diff] [review]:
-----------------------------------------------------------------
Let's err on the side of caution and ask Rebecca if a normal data review process is required for a probe expiration extension.
Attachment #8938374 -
Flags: review?(rweiss)
Attachment #8938374 -
Flags: review?(past)
Attachment #8938374 -
Flags: review+
Assignee | ||
Comment 4•7 years ago
|
||
liuche, could you take the data review here? Thanks!
Flags: needinfo?(florian) → needinfo?(liuche)
Reporter | ||
Comment 5•7 years ago
|
||
Comment on attachment 8938374 [details] [diff] [review]
bump to 65
Try confirms that this fixes the issues with this test.
Attachment #8938374 -
Flags: feedback+
Comment 6•7 years ago
|
||
Comment on attachment 8938374 [details] [diff] [review]
bump to 65
Review of attachment 8938374 [details] [diff] [review]:
-----------------------------------------------------------------
Yes! Approving this extension of the probe.
Attachment #8938374 -
Flags: review?(rweiss) → review+
Updated•7 years ago
|
Flags: needinfo?(liuche)
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/a64b8a0a98a3
Keep the SEARCH_RESET_RESULT Telemetry probe until Firefox 65. r=past, data-r=liuche
Comment 8•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 59
Reporter | ||
Updated•7 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•