Closed
Bug 1343598
Opened 7 years ago
Closed 7 years ago
missing files in beetmover for staging 53.0b1
Categories
(Release Engineering :: Release Automation: Other, enhancement)
Release Engineering
Release Automation: Other
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Unassigned)
References
Details
Attachments
(1 file)
906 bytes,
patch
|
mtabara
:
review+
|
Details | Diff | Splinter Review |
Not sure if they're also missing from nightly. from https://bugzilla.mozilla.org/show_bug.cgi?id=1343393#c10 : 3. At the en-US locale, for both Fennecx86 and Fennec: a) we seem to be missing: File fennec-52.0b8.en-US.android-arm.crashreporter-symbols-full.zip 453M 20-Feb-2017 21:16 File fennec-52.0b8.en-US.android-arm.crashreporter-symbols.zip 97M 20-Feb-2017 21:17 b) we're missing the /host/bin/{mar,mbsdiff}. I see they're present in beetmover templates but they're not being specified in the upstreamArtifacts, see last fennec staging task[2] [2]: https://tools.taskcluster.net/task-group-inspector/#/QQjEh3q1TOu2liv57iHawg/OzMj6pdsQJSM6H9oKIu7yg?_k=mlp0pv
Reporter | ||
Updated•7 years ago
|
Assignee: nobody → aki
Reporter | ||
Comment 1•7 years ago
|
||
:mtabara - do we also want these for nightly?
Flags: needinfo?(mtabara)
Reporter | ||
Comment 2•7 years ago
|
||
Re: mar and mbsdiff, we upload them for linux* but not for android atm. This seems appropriate to me, but I might be wrong?
Reporter | ||
Comment 3•7 years ago
|
||
We don't seem to upload mar or mbsdiff in the build atm: https://tools.taskcluster.net/task-inspector/#YmvSQYOzRZiJrHWMzJzbCw/0
Comment 4•7 years ago
|
||
It seems that we're missing the {crashreporter-symbols-full,crashreporter-symbols} + {mar,mbsdiff} for both nightly and release[2] as well. [1]: http://archive.mozilla.org/pub/mobile/nightly/2017/01/2017-01-01-03-02-04-mozilla-central-android-api-15/en-US/ [2]: http://archive.mozilla.org/pub/mobile/candidates/52.0b10-candidates/build1/android-api-15/en-US/
Flags: needinfo?(mtabara)
Reporter | ||
Comment 5•7 years ago
|
||
This is related to bug 1316125. We're already intentionally skipping crashsymbols beetmove; mar and mbsdiff are available on the linux release. I think we're ok without these files.
Reporter | ||
Updated•7 years ago
|
Reporter | ||
Updated•7 years ago
|
Assignee: aki → nobody
Reporter | ||
Comment 6•7 years ago
|
||
wontfix for now?
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Reporter | ||
Comment 7•7 years ago
|
||
12:40 <mtabara> so how about we skip the {mar,mbsdiff} and add just the crashreporter-symbols.zip for mobile. that seems to lign up with all requirements 12:40 <mtabara> for now ... 12:41 <aki> that would just be adding https://hg.mozilla.org/projects/jamun/file/tip/taskcluster/taskgraph/transforms/beetmover.py#l22 to https://hg.mozilla.org/projects/jamun/file/tip/taskcluster/taskgraph/transforms/beetmover.py#l47 i believe 12:41 <mtabara> that is correcy 12:41 <mtabara> *correct Let's add the crashreporter-symbols.zip (not full)
Reporter | ||
Comment 8•7 years ago
|
||
Attachment #8842578 -
Flags: review?(mtabara)
Updated•7 years ago
|
Attachment #8842578 -
Flags: review?(mtabara) → review+
Reporter | ||
Comment 9•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ba2210b23ab3c96094450b9be4bb05e2089dba20 bug 1343598 - add crashreporter symbols to mobile beetmover tasks. r=mtabara a=release DONTBUILD
Reporter | ||
Comment 10•7 years ago
|
||
https://hg.mozilla.org/projects/jamun/rev/c15b51a28204590aab5862c48971d349e5b4a5c5 bug 1343598 - add crashreporter symbols to mobile beetmover tasks. r=mtabara a=release
Reporter | ||
Comment 11•7 years ago
|
||
We still need to land on m-c and uplift to m-a. Testing a new fennec staging beta.
Reporter | ||
Comment 12•7 years ago
|
||
Uplifted to aurora, and we have the crashreporter symbols in en-US.
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
Comment 13•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ba2210b23ab3
You need to log in
before you can comment on or make changes to this bug.
Description
•