Closed
Bug 946848
Opened 11 years ago
Closed 11 years ago
Make TBPL display 'mochitest-debug-N' chunks with the chunk number
Categories
(Tree Management Graveyard :: TBPL, defect)
Tree Management Graveyard
TBPL
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jgriffin, Assigned: jgriffin)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
emorley
:
review+
RyanVM
:
feedback+
|
Details | Diff | Splinter Review |
We've introduced a new suite name, mochitest-debug, for debug mochitests run on B2G. We've done this to allow us to use different chunk numbers in opt vs debug builds.
Currently, these are running only on cedar and pine, where each chunk shows up as "M". We should make TBPL recognize mochitest-debug, so the chunks will show up as M(1, 2, ...).
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8343217 -
Flags: review?(emorley)
Comment 2•11 years ago
|
||
Can't we just put a (-debug) on the mochitest condition (or something to that effect)?
Comment 3•11 years ago
|
||
Confirmed that this does the trick:
(?:mochitest(?:-debug)?|reftest|crashtest|robocop|androidx86-set|browser-chrome)\-([0-9]+)
Assignee | ||
Comment 4•11 years ago
|
||
Attachment #8343259 -
Flags: review?(emorley)
Assignee | ||
Updated•11 years ago
|
Attachment #8343217 -
Attachment is obsolete: true
Attachment #8343217 -
Flags: review?(emorley)
Assignee | ||
Updated•11 years ago
|
Attachment #8343259 -
Flags: review?(emorley) → review?(ryanvm)
Comment 5•11 years ago
|
||
Comment on attachment 8343259 [details] [diff] [review]
Recognize mochitest-debug chunks,
LGTM, but Ed really should review this. He'll be back soon.
Attachment #8343259 -
Flags: review?(ryanvm)
Attachment #8343259 -
Flags: review?(emorley)
Attachment #8343259 -
Flags: feedback+
Comment 6•11 years ago
|
||
Comment on attachment 8343259 [details] [diff] [review]
Recognize mochitest-debug chunks,
(Have been PTO since Fri eve 29th Nov, hence delay :-))
lgtm, and tests fine locally; thank you :-)
Attachment #8343259 -
Flags: review?(emorley) → review+
Assignee | ||
Comment 7•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Updated•10 years ago
|
Product: Webtools → Tree Management
Updated•10 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•