Closed Bug 607660 Opened 14 years ago Closed 14 years ago

mobile buildsymbols config tweaks

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Assigned: mozilla)

Details

Attachments

(3 files, 3 obsolete files)

1) Disable Maemo uploadsymbols for depend builds. Maybe disable |make uploadsymbols| on all mobile platforms for depend builds
2) can we built crashsymbols zips for tinderbox-builds/ (talos/unittest runs) without running uploadsymbols? I think so, but it'll take a logic change.
3) Look at Android -- I believe we're only running make buildsymbols on nightlies. If we can do that on depend builds, but not make uploadsymbols, that may help in talos/unittests
4) Android nothumb nightly buildsymbols?

The reason for not running make uploadsymbols is for disk space reasons.

Not assigning to myself directly atm in case someone else wants this, but if it's sitting there in triage during tomorrow's call, feel free to assign this to me.
(In reply to comment #0)
> 2) can we built crashsymbols zips for tinderbox-builds/ (talos/unittest runs)
> without running uploadsymbols? I think so, but it'll take a logic change.
> 3) Look at Android -- I believe we're only running make buildsymbols on
> nightlies. If we can do that on depend builds, but not make uploadsymbols, that
> may help in talos/unittests

Yeah, "buildsymbols" creates the zip files, and they'll get uploaded with "upload". "uploadsymbols" puts them on the symbol server.

> The reason for not running make uploadsymbols is for disk space reasons.

That's one reason, the other reason is that our cleanup script tries to only keep 30 days worth of nightly symbols, and it can't tell the difference between hourly and nightly builds, so it will only wind up keeping 30 total hourly+nightly builds, which means that your nightly symbols will not stick around long.
(In reply to comment #0)
> 4) Android nothumb nightly buildsymbols?

If we're going to have crash reporter turned on for nothumb builds, we need the symbols to be uploaded. The top 12 crashes on crash-stats are all SIGILLs from armv6 devices, none of which have symbols for their stacks.
(In reply to comment #3)
> Not assigning to myself directly atm in case someone else wants this, but if
> it's sitting there in triage during tomorrow's call, feel free to assign this
> to me.

Your wish has been granted! :-)
Assignee: nobody → aki
Status: NEW → ASSIGNED
For buildbot-0.8.x and buildbot-0.7.x.

Since we want to generate symbols more often than we run |make uploadsymbols|, I'm creating a generateSymbols flag to go along with uploadSymbols, which both default to False.

Uploading of symbols should only happen if the branch is set to upload_mobile_symbols=True and the platform is set to generate_symbols=True and the build is nightly=True (buildbot-0.8.x only), which should prevent the uploading of symbols on all of our project branches.

It also turns symbol generation on for Android nothumb builds.
This adds the generate/upload symbols logic to misc.py generateMobileBranchObjects, and factory.py {Mobile,Maemo}BuildFactory for 08x.
Attachment #487690 - Attachment is patch: true
Attachment #487690 - Attachment mime type: application/octet-stream → text/plain
This adds the generate/upload symbols logic to the 07x factories.
Tested against 08x and 07x.
Attachment #487687 - Attachment is obsolete: true
Attachment #488932 - Flags: review?(jhford)
Attachment #487692 - Attachment is obsolete: true
Attachment #488933 - Flags: review?(jhford)
See above comments for more info.
Attachment #487690 - Attachment is obsolete: true
Attachment #488935 - Flags: review?(jhford)
Attachment #488933 - Flags: review?(jhford) → review+
Attachment #488935 - Flags: review?(jhford) → review+
Attachment #488932 - Flags: review?(jhford) → review+
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: