Closed Bug 1269004 Opened 8 years ago Closed 8 years ago

Intermittent ecma_7/SIMD/load.js | (args: "--ion-eager --ion-offthread-compile=off") | (TIMEOUT)

Categories

(Core :: JavaScript Engine, defect)

49 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox49 --- fixed

People

(Reporter: KWierso, Assigned: bbouvier)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

This test just needs to be broken into parts.
Flags: needinfo?(bbouvier)
Sheriffs: note that plenty of these oranges are actually not related to this particular test case failing, and are other failures. I've picked randomly a few of them and looked at the output log, and the ecma_7/SIMD/load.js was not even mentioned there. It might be an issue of the auto-classifier, maybe?
Flags: needinfo?(bbouvier) → needinfo?(sheriffs)
A needinfo on a watchable alias like sheriffs@ doesn't actually do anything (other than the exact same thing as cc'ing that alias).

The auto-classifier doesn't actually *do* anything, which is a pretty big problem with its existence, since people think it does things, when exactly and entirely what it does is say "I once saw a push where this failed, and was retriggered and the retrigger didn't hit the same thing."

What you are seeing there is fitzgen loading one failed job, clicking a suggested bug, loading another failed job, clicking a suggested bug there, and once he has clicked every suggested bug for every failure, clicking the Save button once, which stars every one of his failed jobs on his try push as every one of those bugs.
Flags: needinfo?(sheriffs)
Comment on attachment 8755442 [details]
MozReview Request: Bug 1269004: Split the SIMD load tests into several smaller ones; r?lth

https://reviewboard.mozilla.org/r/54612/#review51200

Rubber stamp - I assume you only refactored.  (MozReview fail: I'm not allowed to see the old file because it was deleted.)
Attachment #8755442 - Flags: review?(lhansen)
Assignee: nobody → bbouvier
Comment on attachment 8755442 [details]
MozReview Request: Bug 1269004: Split the SIMD load tests into several smaller ones; r?lth

https://reviewboard.mozilla.org/r/54612/#review52878
Attachment #8755442 - Flags: review+
(In reply to Lars T Hansen [:lth] from comment #7)
> Comment on attachment 8755442 [details]
> MozReview Request: Bug 1269004: Split the SIMD load tests into several
> smaller ones; r?lth
> 
> https://reviewboard.mozilla.org/r/54612/#review51200
> 
> Rubber stamp - I assume you only refactored.  (MozReview fail: I'm not
> allowed to see the old file because it was deleted.)

That is correct, this is just code motion. Deleted files should appear after bug 1114997 is fixed. Thanks!
https://hg.mozilla.org/mozilla-central/rev/0600bf9b878c
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: