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)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Unassigned)

References

Details

Attachments

(1 file)

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
Assignee: nobody → aki
:mtabara - do we also want these for nightly?
Flags: needinfo?(mtabara)
Re: mar and mbsdiff, we upload them for linux* but not for android atm. This seems appropriate to me, but I might be wrong?
We don't seem to upload mar or mbsdiff in the build atm: https://tools.taskcluster.net/task-inspector/#YmvSQYOzRZiJrHWMzJzbCw/0
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)
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.
No longer blocks: 1343393
Depends on: 1316125
Assignee: aki → nobody
wontfix for now?
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
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)
Blocks: 1343393
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Attached patch crashreporter.diff (deleted) — Splinter Review
Attachment #8842578 - Flags: review?(mtabara)
Attachment #8842578 - Flags: review?(mtabara) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/ba2210b23ab3c96094450b9be4bb05e2089dba20 bug 1343598 - add crashreporter symbols to mobile beetmover tasks. r=mtabara a=release DONTBUILD
https://hg.mozilla.org/projects/jamun/rev/c15b51a28204590aab5862c48971d349e5b4a5c5 bug 1343598 - add crashreporter symbols to mobile beetmover tasks. r=mtabara a=release
We still need to land on m-c and uplift to m-a. Testing a new fennec staging beta.
Uplifted to aurora, and we have the crashreporter symbols in en-US.
Status: REOPENED → RESOLVED
Closed: 8 years ago8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: