Closed Bug 489019 Opened 15 years ago Closed 15 years ago

Upload symbols to Socorro.

Categories

(Camino Graveyard :: General, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Camino2.0

People

(Reporter: stuart.morgan+bugzilla, Assigned: stuart.morgan+bugzilla)

References

Details

Attachments

(1 file)

We have (some of) our symbols being generated now, so we need to post them.

Ted, is it just as simple as feeding all our dSYM files and all the core dylibs we ship into symbolstore.py, zipping it up, then running the uploader script? Do we need to follow the same enclosing folder hierarchy as the buildsymbols target does when zipping?

(Apparently I should have looked into this more before putting target-level scripts to generate breakpad symbol files, since this does it for us...)
I don't know if you can use "make buildsymbols" + "make uploadsymbols" in the top-level Makefile, they're what we use. "buildsymbols" will point symbolstore.py at dist/bin, run dsymutil on all the binaries it finds, then run dump_syms and zip the proper directory structure.
http://mxr.mozilla.org/mozilla-central/source/Makefile.in#180

If not, well, then just call symbolstore.py just like that, and zip the contents of the directory.
Note to self: flip SendAndExit back to YES once we do this; we've turned it off until we have symbols so there's some way to get useful reports.
Target Milestone: --- → Camino2.0
Blocks: 488518
Actually we'll remove SendAndExit, per bug 488518 comment 1.
No longer blocks: 488518
Ted, do you know what the process is for getting caminobld set up as a valid account (with key) on dm-symbolpush01.mozilla.org, and how to find out what our SYMBOL_SERVER_PATH should be?
Depends on: 488512
Just file a Server Ops bug asking for that, and they can make it whatever you want. Firefox symbols go to /mnt/netapp/breakpad/symbols_ffx, Thunderbird are in symbols_tbrd, etc. CC aravind on the bug, as he's taken care of this in the past.
(In reply to comment #5)
> Just file a Server Ops bug asking for that

Filed bug 490184; thanks, Ted.
So, we have one more problem here.  Everyone else's generation+upload is done in post-mozilla-rel.pl (via commands to "make buildsymbols" + "make uploadsymbols" as noted above), and post-mozilla-rel.pl has this awesome logic ("$cachebuild") to determine whether we're a nightly/release or just any old build.  If we're a nightly/release, there be symbols; otherwise, no symbols.

After staring at several thousand lines of perl across a couple of files for a couple of hours (at, admittedly, a late hour), I can't see a way for us to get that information ("are we a nightly/release build?") into the environment where we can then make our uploadsymbols pseudo-target ifdefed on only then, rather than clogging the server with worthless symbols every 20 minutes all day.
(The part 2 of that comment is that symbols only get uploaded for successful builds--those that succeed in unify and that pass tests--too.)
Part I: this makes our symbol build and upload into separate targets. That way we can change post-mozilla-rel.pl in part II to make our build and upload targets instead of the core versions.

This also addresses bug 488512 comment 16, adding --vcs-info (and doing a bit of fixup since our paths aren't quite what the script expects).
Attachment #375372 - Flags: review?(alqahira)
Comment on attachment 375372 [details] [diff] [review]
Camino build changes [landed]

r=ardissone
Attachment #375372 - Flags: review?(alqahira) → review+
Comment on attachment 375372 [details] [diff] [review]
Camino build changes [landed]

Landed on cvs trunk.
Attachment #375372 - Attachment description: Camino build changes → Camino build changes [landed]
I checked in a follow-up fix to Makefile.in; I was feed the wrong path in to upload_symbols.sh
Ted, we appear to be missing some final bit of magic. We got symbols uploaded, e.g.:
  http://symbols.mozilla.org/symbols_camino/Camino/419A401095604F04A317E2105CEFC8F10/Camino.sym
and then I crashed that build:
  http://crash-stats.mozilla.com/report/index/9e20b6a1-4e62-45cf-81ed-703992090501?p=1
But as you can see from the crash report the symbols aren't being used for the Camino binary.

Do you know what we are missing? Something to do with the new symbols_camino path, perhaps?

(The only other difference I know of between our symbols and others is that we have two symbol manifests (if that's the term; the output of symbolstore.py) with a different naming schemee, but I assume that is purely for the benefit of humans rather than Socorro)
The symbols.txt files are purely for our symbol cleanup scripts. The processor looks symbols up by filename+UUID (you can verify they match in the Modules tab). You should ask aravind if he added your new path to the processor search path in bug 490184.
Everything is working, and getting our local tinderbox script changes landed is now bug 491642, so I think we can call this FIXED.

Ted, thanks so much for all your help! We couldn't have gotten here with your patience with all my n00b questions ;)
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
I'm just glad to get everyone off that *other* crash reporting system finally!
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: