Closed
Bug 1422735
Opened 6 years ago
Closed 6 years ago
Change symbol upload URL from Socorro to Tecken
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox-esr5258+ fixed, firefox57 wontfix, firefox58 fixed, firefox59 fixed)
RESOLVED
FIXED
mozilla59
People
(Reporter: ted, Assigned: ted)
References
Details
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
nalexander
:
review+
gchang
:
approval-mozilla-beta+
jcristau
:
approval-mozilla-release-
gchang
:
approval-mozilla-esr52+
|
Details |
peterbe has split symbol handling out of Socorro into a standalone service called Tecken: https://github.com/mozilla-services/tecken/ It's live serving symbols.mozilla.org right now, but we're still uploading symbols via Socorro's symbol upload API. We'd like to switch over uploads to Tecken. This should be as simple as just changing the upload URL to https://symbols.mozilla.org/upload/ here: https://dxr.mozilla.org/mozilla-central/rev/574f4f58fe09dd590ea892406e237318c31705b4/toolkit/crashreporter/tools/upload_symbols.py#28
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → ted
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•6 years ago
|
||
The patch in bug 1422740 got a little more complicated than I expected and it doesn't really block this change, so I just wrote the one-line patch to change the upload URL instead. This should be simple to land whenever we're ready and easy to uplift across branches.
No longer depends on: 1422740
Comment 3•6 years ago
|
||
mozreview-review |
Comment on attachment 8934934 [details] bug 1422735 - Change symbol upload URL from Socorro to Tecken. https://reviewboard.mozilla.org/r/205864/#review211648 Technically fine, and I'm confident you'll follow whatever policy is required. Verified URL at https://github.com/mozilla-services/tecken/blob/8eb9143276fbdc422ed27e1990a19cf1d9bcdb76/docs/upload.rst.
Attachment #8934934 -
Flags: review+
Updated•6 years ago
|
Attachment #8934934 -
Flags: review?(core-build-config-reviews)
Pushed by tmielczarek@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/04f52fb15e76 Change symbol upload URL from Socorro to Tecken. r=nalexander
Comment 5•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/04f52fb15e76
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox59:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Assignee | ||
Comment 6•6 years ago
|
||
When we originally landed this patch we broke nightly symbol uploads (bug 1424236) because the tokens that were migrated from Socorro to Tecken didn't get their permissions migrated with them, so Tecken refused the uploads. We worked around this on central by landing my patches in bug 1422740 to use a token from a Taskcluster secret, and I generated a new token in Tecken to use there. Since then peterbe fixed the migrated tokens (bug 1424326), so we'd like to land this small patch on all other branches so that all symbol uploads are going through Tecken.
Assignee | ||
Comment 7•6 years ago
|
||
Comment on attachment 8934934 [details] bug 1422735 - Change symbol upload URL from Socorro to Tecken. Per my previous comment, we've had symbol uploads going through Tecken on central for about a month now, and we'd like to switch other branches over so we can eventually EOL the Socorro symbol upload API. This patch is a single line that changes the symbol upload URL. It does not affect the shipped builds, but could impact crash reporting if symbols fail to upload. We do not expect any issues from switching the upload URL.
Attachment #8934934 -
Flags: approval-mozilla-release?
Attachment #8934934 -
Flags: approval-mozilla-esr52?
Attachment #8934934 -
Flags: approval-mozilla-beta?
Comment 8•6 years ago
|
||
Comment on attachment 8934934 [details] bug 1422735 - Change symbol upload URL from Socorro to Tecken. Take this to change symbol upload URL. Beta58+ & ESR52+.
Attachment #8934934 -
Flags: approval-mozilla-esr52?
Attachment #8934934 -
Flags: approval-mozilla-esr52+
Attachment #8934934 -
Flags: approval-mozilla-beta?
Attachment #8934934 -
Flags: approval-mozilla-beta+
Updated•6 years ago
|
status-firefox57:
--- → affected
status-firefox58:
--- → affected
status-firefox-esr52:
--- → affected
tracking-firefox-esr52:
--- → 58+
Comment 9•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/3a94f3140a14
Comment 10•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-esr52/rev/b88ef76f5687
Updated•6 years ago
|
Comment 11•6 years ago
|
||
Comment on attachment 8934934 [details] bug 1422735 - Change symbol upload URL from Socorro to Tecken. 58 is on mozilla-release now, so we won't be making that change to 57.
Attachment #8934934 -
Flags: approval-mozilla-release? → approval-mozilla-release-
Assignee | ||
Comment 12•6 years ago
|
||
Gotcha, thanks!
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•