Closed Bug 1171218 Opened 6 years ago Closed 6 years ago

Build symbols not working correctly in Thunderbird 38


(Thunderbird :: Build Config, defect)

38 Branch
Not set


(thunderbird41 fixed, thunderbird42 fixed, thunderbird43 fixed, thunderbird_esr38 fixed)

Thunderbird 43.0
Tracking Status
thunderbird41 --- fixed
thunderbird42 --- fixed
thunderbird43 --- fixed
thunderbird_esr38 --- fixed


(Reporter: rkent, Assigned: rkent)




(1 file)

See bug 1085557.

Now I am getting a failure in comm-esr38 builds due to differences in build files associated with this line:

export SOCORRO_SYMBOL_UPLOAD_TOKEN_FILE=/builds/crash-stats-api.token

Looking back at the history of this, these lines have been slowly added in comm-central as ad-hoc reactions to build failures. Also looking at crash stats, something is clearly wrong with symbol uploads, we don't get links to lines in comm-central from at least the Windows crashes.
Attached patch Add win-commonSplinter Review
I believe this is what we need for Windows, in comm-central and other repos.

For comm-esr38 we also need to port some of the Linux and OSX patches as well.
Assignee: nobody → rkent
Attachment #8614952 - Flags: review?(Pidgeot18)
You don't actually want the symbol upload token defined in your mozconfig unless you have a valid token in that file on the builder. Otherwise your symbol uploads are not going to work.
I was under the impression that Thunderbird had its own separate builders, but I'm told Thunderbird builds use the same build pool as Firefox, so putting the same path to the auth token in mozconfigs should be fine.

If you have Linux Thunderbird builds using mozharness+mock you'll need to make sure the mock config copies the token into the mock environment, but other than that it should work fine.
FTR: according to bug 1170253 comment 8, the firefox token file already present should work as we're using the same builder pool.
Oh, you already wrote that, sorry.
Comment on attachment 8614952 [details] [diff] [review]
Add win-common

>diff --git a/build/ b/build/
>new file mode 100644
>--- /dev/null
>+++ b/build/
>@@ -0,0 +1,16 @@
>+# This Source Code Form is subject to the terms of the Mozilla Public
>+# License, v. 2.0. If a copy of the MPL was not distributed with this
>+# file, You can obtain one at
>+if [ "x$IS_NIGHTLY" = "xyes" ]; then
>+  # Some nightlies (eg: Mulet) don't want these set.
>+# Some builds (eg: Mulet) don't want the installer, so only set this if it
>+# hasn't already been set.

Everything before here is going to be ignored at best, or will cause bustage.

>+export SOCORRO_SYMBOL_UPLOAD_TOKEN_FILE=c:/builds/crash-stats-api.token

You only need this line.
Are you adding that because the comparison tool demands it ?
(In reply to Nick Thomas [:nthomas] from comment #8)
> Are you adding that because the comparison tool demands it ?

Yes. I'm not sure of the how or why of the comparison tool, but we'll get errors if we have the same file with different content.

Ignoring is OK, breakage is bad. Do you know which it is?
It's probably ignore. Those environment variables are used when the automation calls 'mach build' in mozharness, and AFAIK Thunderbird isn't doing that. For reference some to build system for this is
Yes. Without the export of SOCORRO_SYMBOL_UPLOAD_TOKEN_FILE the symbols are uploaded to the old location, and presumably the sync from old to new is disabled now. I've fixed up 38.2.0, see bug 1194989 comment #3.
Comment on attachment 8614952 [details] [diff] [review]
Add win-common

Review of attachment 8614952 [details] [diff] [review]:

Stealing this - looks ok to me so r=mkmelin
Attachment #8614952 - Flags: review?(Pidgeot18) → review+
Blocks: 1194989
Duplicate of this bug: 1197490
Go ahead and land this patch if you want comm-central and comm-aurora nightlies back.
Bug 1171218 - Build symbols not working correctly in Thunderbird 38. r=mkmelin a=bustage fix on CLOSED TREE
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 43.0
Needs uplift to all kinds of places?
Flags: needinfo?(rkent)
Comment on attachment 8614952 [details] [diff] [review]
Add win-common

Yes presumably this needs uplift. It would be good to see at least one working nightly before we do that.
Flags: needinfo?(rkent)
Attachment #8614952 - Flags: approval-comm-esr38?
Attachment #8614952 - Flags: approval-comm-beta?
Attachment #8614952 - Flags: approval-comm-aurora?
There should be a green Window nightlies on comm-central today. We actually could have verified this earlier though, as the cause of the orange occurs after the symbol upload. It's been working:

Uploading symbol file "dist/" to ""
Attempt 1 of 5...
Uploaded successfully!


Would be very helfpul to uplift this, particularly for 41.0b1 build2 and 38.3.0.
Comment on attachment 8614952 [details] [diff] [review]
Add win-common

I'm going to plus the approvals here, we should definitely use the new system asap.
Attachment #8614952 - Flags: approval-comm-esr38?
Attachment #8614952 - Flags: approval-comm-esr38+
Attachment #8614952 - Flags: approval-comm-beta?
Attachment #8614952 - Flags: approval-comm-beta+
Attachment #8614952 - Flags: approval-comm-aurora?
Attachment #8614952 - Flags: approval-comm-aurora+
(using checkin-needed for branch checkins is confusing, I think you just have to look at the flags for what approved but not fixed)
Keywords: checkin-needed
This seem to have caused test bustage on comm-esr38:

> Linux opt Build (B)
> TEST-UNEXPECTED-FAIL | | build file copies are not in sync
> TEST-INFO | | file(s) found in: /builds/slave/tb-c-esr38-lx-0000000000000000/build/build
> TEST-INFO | | differ from their originals in: /builds/slave/tb-c-esr38-lx-0000000000000000/build/mozilla/build
> TEST-INFO | | differing file: ./
Looks like a typo from a previous checkin to me
I'm not a programmer, but shouldn't that be if and not fi
No. "fi" closes the "if" clause (and the line you mention wasn't changed)
You need to log in before you can comment on or make changes to this bug.