Closed
Bug 905822
Opened 11 years ago
Closed 11 years ago
Annotate manifests with run-sequentially
Categories
(MailNews Core :: Testing Infrastructure, defect)
MailNews Core
Testing Infrastructure
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 26.0
People
(Reporter: mihneadb, Assigned: mihneadb)
Details
Attachments
(1 file)
9.76 KB,
patch
|
jcranmer
:
review+
|
Details | Diff | Splinter Review |
Doing this so the new parxpc patch doesn't cause bustage.
Assignee | ||
Updated•11 years ago
|
Summary: Annotate manifests with run0-sequentially → Annotate manifests with run-sequentially
Assignee | ||
Comment 1•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → mihneadb
Status: NEW → ASSIGNED
Assignee | ||
Updated•11 years ago
|
Attachment #790976 -
Flags: review?(Pidgeot18)
Comment 2•11 years ago
|
||
Comment on attachment 790976 [details] [diff] [review] Annotate manifests with run-sequentially Review of attachment 790976 [details] [diff] [review]: ----------------------------------------------------------------- Okay, so running tests in parallel right now requires using mach, which is broken as hell for comm-central, so this is not an imminently-needed bustage fix. Still, bug 660788 could land soonish, so it's good to have all of this in so we don't make our xpcshell test coverage go down the drain again. Without this patch, I get 184 failures out of 287 for mailnews, including some epically complete bustage in at least news and IMAP, and sporadic failures in compose and gloda it looks like (hard to classify scrolling pages of failing tests).
Attachment #790976 -
Flags: review?(Pidgeot18) → review+
Updated•11 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 3•11 years ago
|
||
(In reply to Joshua Cranmer [:jcranmer] from comment #2) > Comment on attachment 790976 [details] [diff] [review] > Annotate manifests with run-sequentially > > Review of attachment 790976 [details] [diff] [review]: > ----------------------------------------------------------------- > > Okay, so running tests in parallel right now requires using mach, which is > broken as hell for comm-central, so this is not an imminently-needed bustage > fix. Still, bug 660788 could land soonish, so it's good to have all of this > in so we don't make our xpcshell test coverage go down the drain again. > > Without this patch, I get 184 failures out of 287 for mailnews, including > some epically complete bustage in at least news and IMAP, and sporadic > failures in compose and gloda it looks like (hard to classify scrolling > pages of failing tests). So what do you get *with* the patch? :)
Comment 4•11 years ago
|
||
Only the failures of bug 901959. :-)
Assignee | ||
Comment 5•11 years ago
|
||
Ah, sweet, so my patch fixes the problem.
Comment 6•11 years ago
|
||
https://hg.mozilla.org/comm-central/rev/e99c2bc2eda1
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 26.0
You need to log in
before you can comment on or make changes to this bug.
Description
•