Closed Bug 816581 Opened 12 years ago Closed 12 years ago

TBPL should recognise "^DMError:"

Categories

(Tree Management Graveyard :: TBPL, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Would mean we can star bug 814481 and friends. We should probably add some more exceptions types later (MarionetteError,...)
Attached patch Patch v1 (deleted) — Splinter Review
Just to check: are there any(/many) expected cases where DMError appears in the log, but it is non-fatal?
Attachment #686644 - Flags: review?(wlachance)
Comment on attachment 686644 [details] [diff] [review] Patch v1 I am not aware of any instances of where we'd print DMError under normal circumstances. Maybe double check this with :jmaher just to be sure.
Attachment #686644 - Flags: review?(wlachance) → review+
Seems ok locally; next best way to check is on tbpl-dev tbh: https://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/71a17b08d422
Depends on: 817713
In production.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
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: