Closed
Bug 1394083
Opened 7 years ago
Closed 7 years ago
Ship the Follow-on search telemetry system add-on v0.9.3 via Go Faster to 55 users
Categories
(Firefox :: Search, enhancement, P1)
Tracking
()
VERIFIED
FIXED
People
(Reporter: standard8, Unassigned)
Details
(Whiteboard: [go-faster-system-addon][fxsearch])
Attachments
(2 files, 1 obsolete file)
We want to ship the v0.9.3 of the system add-on to users of FF 55. This will: - Add additional Yahoo domains - Correct Google follow-tracking, regressed by the removal of Google Instant Search See the 0.9.2 and 0.9.3 release notes here: https://github.com/mozilla/followonsearch/releases
Reporter | ||
Comment 1•7 years ago
|
||
Jason/wezhou, please can one of you sign this file for shipping via Go Faster. Thanks!
Flags: needinfo?(wezhou)
Flags: needinfo?(jthomas)
The attachment 8901424 [details] is not properly zipped, and although "unzip" command was able to unzip it with warnings, I can't tell for sure the unzipped files are intact. Thus I'd recommend re-zipping it again with proper format and try again.
$ unzip attachment.8901424.xpi
Archive: attachment.8901424.xpi
warning [attachment.8901424.xpi]: 17017 extra bytes at beginning or within zipfile
(attempting to process anyway)
error [attachment.8901424.xpi]: reported length of central directory is
-17017 bytes too long (Atari STZip zipfile? J.H.Holm ZIPSPLIT 1.1
zipfile?). Compensating...
extracting: chrome.manifest
extracting: bootstrap.js
extracting: chrome/content/followonsearch-fs.js
extracting: install.rdf
Flags: needinfo?(wezhou)
Flags: needinfo?(jthomas)
Comment 3•7 years ago
|
||
Attachment #8901424 -
Attachment is obsolete: true
Comment 5•7 years ago
|
||
I've verified the fix is working.
Updated•7 years ago
|
QA Contact: amasresha
Comment 6•7 years ago
|
||
Abe, Can you please provide sign off once you have verified the update? Thanks!
Comment 7•7 years ago
|
||
We have tested this on Firefox 55.0.3 with Follow-on Search Telemetry 0.9.3 add-on. The following operating systems are used for testing: Windows-10 64bits, Windows 7-32bits, Ubuntu 16.04 64bits, and OSX10.12. All of our test runs show green.
Comment 8•7 years ago
|
||
(In reply to shgupta@mozilla.com from comment #6) > Abe, Can you please provide sign off once you have verified the update? > Thanks! Sign-off is sent now. I will mark this bug as verified-fixed after the developer closes it as resolved(if needed). Thanks
Reporter | ||
Comment 9•7 years ago
|
||
We'll close it once we're actually pushing the system add-on out to FF 55 users :-)
Reporter | ||
Comment 10•7 years ago
|
||
Rehan, please could you put the attached add-on on the Go Faster test channel for 55.0.* ? Maybe just 55.0 -> 55.0.3 to be safe.
Flags: needinfo?(rdalal)
Comment 11•7 years ago
|
||
This was added to the test channel "release-sysaddon" for 55.0.*
Flags: needinfo?(rdalal)
Reporter | ||
Comment 12•7 years ago
|
||
I've tested the update of 0.9.3 of the release-sysaddon on Mac & Windows and it works fine. Rehan, we got the sign-off from relman yesterday, please could you push it to the release channel?
Reporter | ||
Updated•7 years ago
|
Flags: needinfo?(rdalal)
Comment 13•7 years ago
|
||
Hi Rehan, We have QA and Relman sign off. Can you please let us know if you need anything else to push this addon update to 55 release? If not, can this go out today? Thanks Shilpi
Comment 14•7 years ago
|
||
I have updated the rules and they are currently awaiting QE/relman sign off in Balrog.
Flags: needinfo?(rdalal)
Flags: needinfo?(lhenry)
Flags: needinfo?(amasresha)
Comment 16•7 years ago
|
||
This shipped to release (55.*) on Friday.
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(lhenry)
Resolution: --- → FIXED
Comment 17•7 years ago
|
||
Thanks All! Mark, Can you confirm if the update is on the tree? Just want to close the loop on that last item for this bug.
Flags: needinfo?(standard8)
Reporter | ||
Comment 18•7 years ago
|
||
I just did a check and we're updating on Mac/Linux. Marking as verified per comment 8
Status: RESOLVED → VERIFIED
Flags: needinfo?(standard8)
You need to log in
before you can comment on or make changes to this bug.
Description
•