Closed Bug 1159999 Opened 9 years ago Closed 9 years ago

Intermittent + trap 'echo '\''TEST-UNEXPECTED-FAIL | autospider.sh 3h timeout | ignore later failures'\'' >&2; exit 1' INT

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox38 --- unaffected
firefox39 --- unaffected
firefox40 --- fixed
firefox-esr31 --- unaffected

People

(Reporter: philor, Assigned: sfink)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

Not sure that output precisely as planned.
Oh, is TH picking up on that? I know it's ugly. It's because I run the whole shell script with -x to echo out everything it's doing.

I guess I'll undo that around that line.
Assignee: nobody → sphink
Status: NEW → ASSIGNED
Comment on attachment 8599680 [details] [diff] [review]
Suppress command echo for timeout failure setup

Review of attachment 8599680 [details] [diff] [review]:
-----------------------------------------------------------------

IMO, trivial enough to not need a review. Or a try push, but I'll see whether a sheriff agrees.
Attachment #8599680 - Flags: review+
Attachment #8599680 - Flags: checkin?
Keywords: checkin-needed
(In reply to Steve Fink [:sfink, :s:] from comment #5)
> Comment on attachment 8599680 [details] [diff] [review]
> Suppress command echo for timeout failure setup
> 
> Review of attachment 8599680 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> IMO, trivial enough to not need a review. Or a try push, but I'll see
> whether a sheriff agrees.

sorry had to back this out steve for test failures like https://treeherder.mozilla.org/logviewer.html#?job_id=9440666&repo=mozilla-inbound
Flags: needinfo?(sphink)
The failures that this was backed out for pre-date this patch landing. I'll re-land.
Flags: needinfo?(sphink)
https://hg.mozilla.org/mozilla-central/rev/09389263844e
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Attachment #8599680 - Flags: checkin? → checkin+
You need to log in before you can comment on or make changes to this bug.