Closed Bug 1379868 Opened 7 years ago Closed 7 years ago

Disable Mochitest-gl under Windows 32-bit non-e10s mode.

Categories

(Core :: Graphics: CanvasWebGL, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla56
Tracking Status
firefox56 --- fixed

People

(Reporter: cleu, Assigned: jmaher)

References

Details

Attachments

(3 files)

There are many intermittent mochitest-gl failures which are related to OOM and they are all present under windows 32-bit non-e10s mode only.

Since those failures are caused by architectural limitation and non-e10s mochitests will be disabled by default after 57 is released, it should be OK to disable mochitest-gl under windows 32-bit non-e10s mode.

It will be beneficial for our mochitest stability.
That sounds fine.
Hi Joel,

According to discussion in bug 1298285, we are considering disabling mochitest-gl under win32 non-e10s mode.

I have discussed with :jgilbert and the conclusion is positive.

So I think we can disable it.

It should reduce many intermittent failures in mochitest-gl.
Flags: needinfo?(jmaher)
this is great, do you want this for all branches or on nightly/trunk and let it ride the trains?
Flags: needinfo?(jmaher)
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Attachment #8886239 - Flags: review?(catlee)
Attachment #8886239 - Flags: review?(catlee) → review?(aki)
Attachment #8886239 - Flags: review?(aki) → review+
this is deployed and for win7 we are not running webgl on non-e10s, we still are for win8.
tc jobs are still running; this should take care of that.
Attachment #8886628 - Flags: review?(jmaher)
Comment on attachment 8886628 [details] [diff] [review]
stop running taskcluster windows 7 non-e10s

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

great, nice and simple.
Attachment #8886628 - Flags: review?(jmaher) → review+
Pushed by gbrown@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/94fbfcd285f5
Stop running mochitest-gl on tc Windows 7 non-e10s; r=jmaher
Can we do this for DevEdition builds as well? At least on Beta, they're still running and failing.
I think it should be fine to do this on other branches as well.
https://hg.mozilla.org/mozilla-central/rev/94fbfcd285f5
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: