449621 - symbol upload for tracemonkey builds

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Mike Shaver (:shaver emeritus)]

14:00 < ted> should be easy
14:00 < ted> there's an env var that we set so as to avoid buildid collisions
14:00 < ted> and then you just make it upload like the normal builds do
14:01 < ted> set MOZ_SYMBOLS_EXTRA_BUILDID=tracemonkey
14:01 < ted> or some such
14:03 < ted> shaver: you may want to file a bug on this so it doesn't get 
             forgotten in the ephemeral stream of irc
14:04 < lsblakk> shaver: i can add that env var to the unittest one
14:04 < shaver> yeah, then we just need to do the actual symbol upload

Comment 1

[[e-mail] (:bhearsum)]

Note: we need to modify the "build" Buildbot (hg.m.o/build/buildbot-configs/mozilla2/config.py to upload symbols for our published nightly builds, not the unittest master.

Comment 2

[Lukas Blakk [:lsblakk] use ?needinfo]

The unittest master.cfg Local Environments needs to be changed as well for the tracemonkey-unittest builds.

Although, if you make this change to the automated builds, perhaps there is no need to upload symbols from the unittest build as well.

Comment 3

[[e-mail] (:bhearsum)]

There's no reason at all to upload unittest build symbols, we don't publish those builds and therefore would not get crash reports from them.

Comment 4

[[e-mail] (:bhearsum)]

Attachment: enable symbol upload for tracemonkey nightlies

Comment 5

[(not currently active) Ted Mielczarek]

Comment on attachment 333743 [details] [diff] [review]
enable symbol upload for tracemonkey nightlies

+    'SYMBOL_SERVER_EXTRA_BUILDID': 'tracemonkey',


You want MOZ_SYMBOLS_EXTRA_BUILDID:
http://mxr.mozilla.org/mozilla-central/source/Makefile.in#176

Comment 6

[[e-mail] (:bhearsum)]

Attachment: [checked in] use correct env var

changeset:   231:c36e83943609

Comment 7

[[e-mail] (:bhearsum)]

I'm kicking new nightlies to give this a try.

Comment 8

[[e-mail] (:bhearsum)]

Mac is still broken due to bug 448616. I haven't kicked a Windows build because the slaves are quite busy right now - the regularly scheduled nightly will have symbols, though.