391718 - Enable Breakpad symbol push for win32 XULRunner nightly builds

Status: RESOLVED FIXED

(Webtools Graveyard :: Tinderbox, defect, P2)

Description

[Nick Thomas [:nthomas] (UTC+12)]

bsmedberg says that we probably want to do this. It's gated on both him and Ted (luser) getting back from vacation and having a chance to talk about it.

Comment 1

[(not currently active) Ted Mielczarek]

I probably missed some context on this.  Why do we want to do this?  Last I knew, we weren't accepting crash reports for random XULRunner apps.  We do have bug 381513 for uploading symbol packages to FTP for XULRunner nightlies, so that XR app authors can use them on their own Socorro servers.

Comment 2

[Nick Thomas [:nthomas] (UTC+12)]

Unfortunately I can't find anything in my IRC logs.

Comment 3

[Benjamin Smedberg]

I mainly want it for the symbol server, not for breakpad processing. I also want to provide a package (maybe in the SDK?) of .sym and .pdb files so that XR developers can run their own symbol server, but that's a different issue.

Comment 4

[(not currently active) Ted Mielczarek]

Ah, ok.  We could enable it just for Win32 then, to get the symbol server.  bug 381513 covers uploading the symbol package to FTP in tinderbox.  Is there an xrbld user or something like that?  All the product builds currently upload their symbols as their respective user.  Either way, we'll need an IT bug to get a breakpad/symbols_xr directory created, and mapped to symbols.mozilla.org/xulrunner, then we can easily turn on symbol push.

Comment 5

[Nick Thomas [:nthomas] (UTC+12)]

Is this still win32 only ? 

Comment 6

[(not currently active) Ted Mielczarek]

Might make sense to do Linux too if I get bug 420474 fixed, but Win32 would be the biggest immediate win.

Comment 7

[Nick Thomas [:nthomas] (UTC+12)]

breakpad/symbols_xr already exists, so lets put some nightly symbols in there and see if the symbol can be retrieved.

Comment 8

[Nick Thomas [:nthomas] (UTC+12)]

Attachment: Enable symbol generation and upload for win32 nightlies

Comment 9

[(not currently active) Ted Mielczarek]

Let's set the env vars to enable source server support while we're at it.

Comment 10

[Nick Thomas [:nthomas] (UTC+12)]

Attachment: [checked in] Enable symbol generation and upload for win32 nightlies, and source server

Good point. How's this look ?

Comment 11

[(not currently active) Ted Mielczarek]

Comment on attachment 322958 [details] [diff] [review]
[checked in] Enable symbol generation and upload for win32 nightlies, and source server

Looks good, but should probably persist the source server comment from the other configs.

Comment 12

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 322958 [details] [diff] [review]
[checked in] Enable symbol generation and upload for win32 nightlies, and source server

Checking in tinder-config.pl;
/cvsroot/mozilla/tools/tinderbox-configs/xulrunner/win32/tinder-config.pl,v  <--  tinder-config.pl
new revision: 1.13; previous revision: 1.12
done

with the requested comment and a clobber request.

Comment 13

[Nick Thomas [:nthomas] (UTC+12)]

This seems to have sensible content (and be accessible)

http://symbols.mozilla.org/xulrunner/mangle.pdb/AF7ECC54FD694D589FE0EC4230AC9FE61/mangle.sym

Ted or Benjamin, could you do a quick verification with Visual Studio ?

Comment 14

[Dave Townsend [:mossop]]

(In reply to comment #13)
> This seems to have sensible content (and be accessible)
> 
> http://symbols.mozilla.org/xulrunner/mangle.pdb/AF7ECC54FD694D589FE0EC4230AC9FE61/mangle.sym
> 
> Ted or Benjamin, could you do a quick verification with Visual Studio ?

I can verify that VS loads symbols and source when attaching to McCoy built against the latest nightly SDK. Awesome!