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)

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: 7 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 → ---
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: 7 years ago7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.