Closed
Bug 1343598
Opened 8 years ago
Closed 8 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)
(deleted),
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•8 years ago
|
Assignee: nobody → aki
Reporter | ||
Comment 1•8 years ago
|
||
:mtabara - do we also want these for nightly?
Flags: needinfo?(mtabara)
Reporter | ||
Comment 2•8 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•8 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•8 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•8 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•8 years ago
|
Reporter | ||
Updated•8 years ago
|
Assignee: aki → nobody
Reporter | ||
Comment 6•8 years ago
|
||
wontfix for now?
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Reporter | ||
Comment 7•8 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•8 years ago
|
||
Attachment #8842578 -
Flags: review?(mtabara)
Updated•8 years ago
|
Attachment #8842578 -
Flags: review?(mtabara) → review+
Reporter | ||
Comment 9•8 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•8 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•8 years ago
|
||
We still need to land on m-c and uplift to m-a.
Testing a new fennec staging beta.
Reporter | ||
Comment 12•8 years ago
|
||
Uplifted to aurora, and we have the crashreporter symbols in en-US.
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → FIXED
Comment 13•8 years ago
|
||
bugherder |
You need to log in
before you can comment on or make changes to this bug.
Description
•