608114 - Adapt to symbols upload changes for post-upload command in bug 607951 and friends

Status: RESOLVED FIXED

(Camino Graveyard :: General, defect)

Comment 1

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Per bug 607951, it looks like we'll need to make only 2 changes (on all active branches):

1) 'export'ing the symbols index filename in our Makefile (so it's available to the environment and scripts)

2) setting the 'POST_SYMBOL_UPLOAD_CMD' env variable in all of our tinderconfigs.  We'll need to know the final path to the command that we should use.

In addition,

3) We'll need to ensure that the relevant changes to toolkit's upload-symbols.sh land on 1.9.0 for Camino 2.0.x builds ;)

4) We'll probably want to port changes to the final upload command into the re-upload script generation: http://mxr.mozilla.org/camino/source/camino/Makefile.in#436

Comment 2

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Attachment: Makefile changes

This should do 1 and 4, the Makefile bits.

Comment 3

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

And for the tinderconfigs, the path-value for the new env variable appears to be '/usr/local/bin/post-symbol-upload.py'

I'll upload the tinderconfig diffs (for sanity) later.

Comment 4

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Attachment: 1.9.2 tinder-configs

Stuart, just looking for a sanity-check from you on these two patches; we really only care about cb-x1 (the one with no machine-name in the path) and cb-x4, but I've modified the other configs that had symbols info to keep them updated.

Comment 5

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Attachment: CAMINO_2_0_BRANCH tinder-configs

Comment 6

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 488116 [details] [diff] [review]
CAMINO_2_0_BRANCH tinder-configs

(I've also made this change to the on-machine Cm2.0.5 tinder-config, so that when it gets copied and edited for 2.0.6, it will pick up this change without me forgetting.)

Comment 7

[Stuart Morgan]

Comment on attachment 487518 [details] [diff] [review]
Makefile changes

sr=smorgan

Comment 8

[Stuart Morgan]

Comment on attachment 488115 [details] [diff] [review]
1.9.2 tinder-configs

sr=smorgan

Comment 9

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 487518 [details] [diff] [review]
Makefile changes

http://hg.mozilla.org/camino/rev/c6c46d400f72 and CAMINO_2_0_BRANCH

Comment 10

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Holding off on landing the tinder-configs until there's movement on bug 607951 and/or bug 607946 and/or branch movement on bug 607946.

I don't want to get us into a situation where upload-symbols.sh has been updated on the branches to use the command but the command (or its no-op version) hasn't been installed on the symbols upload server and causes errors when doing symbol upload, which would be a situation where the otherwise no-op tinder-config changes became op.

Comment 11

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Attachment: 1.9.2 tinder-configs

Now with less build failures due to missing ;s

Comment 12

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Attachment: CAMINO_2_0_BRANCH tinder-configs

Now with less build failures due to missing ;s

Comment 13

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Bug 607946 is in on branches, and bug 609270 is also live (the no-op version), so I landed the tinder-config changes on both branches.

http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=&branchtype=match&dir=%2Fmozilla%2Fcamino+%2Fmozilla%2Ftools%2Ftinderbox-configs%2Fcamino&sortby=Date&hours=2&date=explicit&mindate=2010-11-19+00%3A00&maxdate=2010-11-20+00%3A00&cvsroot=%2Fcvsroot

That means we're all ready whenever bug 607951 and bug 607961 happen to land.  It doesn't look like it'll happen while I'm gone, but I'll've definitely forgotten about this by the time I'm back.

We'll need to check tomorrow's nightlies' symbol uploads to make sure they went OK, though.

Comment 14

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

(And I've updated my OS Symbols convenience scripts to fetch and export the symbol index filename and the post-upload command path, too: http://hg.mozilla.org/users/alqahira_ardisson.org/misc/rev/0648e3037b5b  As long as we keep cb-x1's 2.1 tinderconfig updated, I'll pull the right thing [and we can change it to cb-x4 if we need to, but x1 has a shorter path :P ])