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)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jgriffin, Assigned: jgriffin)

References

Details

Attachments

(1 file, 1 obsolete file)

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, ...).
Attached patch Recognize mochitest-debug chunks, (obsolete) (deleted) — Splinter Review
Attachment #8343217 - Flags: review?(emorley)
Can't we just put a (-debug) on the mochitest condition (or something to that effect)?
Confirmed that this does the trick: (?:mochitest(?:-debug)?|reftest|crashtest|robocop|androidx86-set|browser-chrome)\-([0-9]+)
Attachment #8343259 - Flags: review?(emorley)
Attachment #8343217 - Attachment is obsolete: true
Attachment #8343217 - Flags: review?(emorley)
Attachment #8343259 - Flags: review?(emorley) → review?(ryanvm)
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 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+
Blocks: 949255
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
No longer blocks: 949255
Depends on: 949255
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: