Ask for a longer timeout for Android 2.3 conformance tests

RESOLVED FIXED in Firefox 34

Status

()

Core
Canvas: WebGL
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jgilbert, Assigned: jgilbert)

Tracking

unspecified
mozilla36
All
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox34 fixed, firefox35 fixed, firefox36 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

4 years ago
Created attachment 8511437 [details] [diff] [review]
0001-Give-WebGL-conformance-mochis-a-longer-timeout.patch

We're timing out extremely often on Android 2.3.
Let's extend its timeouts.
Attachment #8511437 - Flags: review?(dglastonbury)
(Assignee)

Updated

4 years ago
Blocks: 1087204
(Assignee)

Updated

4 years ago
Blocks: 1078169
(Assignee)

Updated

4 years ago
Blocks: 1085242
(Assignee)

Updated

4 years ago
Blocks: 1081910
(Assignee)

Updated

4 years ago
Blocks: 1089414
(Assignee)

Updated

4 years ago
Blocks: 1085236
(Assignee)

Updated

4 years ago
Blocks: 1084265
(Assignee)

Updated

4 years ago
Blocks: 1088696
(Assignee)

Updated

4 years ago
Blocks: 1088649
(Assignee)

Updated

4 years ago
Blocks: 1081878
(Assignee)

Updated

4 years ago
Blocks: 1086160
(Assignee)

Updated

4 years ago
Blocks: 1062512
(Assignee)

Updated

4 years ago
Blocks: 1066230
Comment on attachment 8511437 [details] [diff] [review]
0001-Give-WebGL-conformance-mochis-a-longer-timeout.patch

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

::: dom/canvas/test/webgl-conformance/mochi-single.html
@@ +246,5 @@
>  
> +var isAndroid2_3 = (DriverInfo.getOS() == DriverInfo.OS.ANDROID &&
> +                    DriverInfo.getOSVersion() < OS_VERSION_ANDROID_ICS);
> +if (isAndroid2_3) {
> +  SimpleTest.requestLongerTimeout(2);

What does the magic 2 mean?
Attachment #8511437 - Flags: review?(dglastonbury) → review+
(Assignee)

Comment 2

4 years ago
(In reply to Dan Glastonbury :djg :kamidphish from comment #1)
> Comment on attachment 8511437 [details] [diff] [review]
> 0001-Give-WebGL-conformance-mochis-a-longer-timeout.patch
> 
> Review of attachment 8511437 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: dom/canvas/test/webgl-conformance/mochi-single.html
> @@ +246,5 @@
> >  
> > +var isAndroid2_3 = (DriverInfo.getOS() == DriverInfo.OS.ANDROID &&
> > +                    DriverInfo.getOSVersion() < OS_VERSION_ANDROID_ICS);
> > +if (isAndroid2_3) {
> > +  SimpleTest.requestLongerTimeout(2);
> 
> What does the magic 2 mean?

Good point. It's a multiple for the standard timeout time. `2` gives us twice as long of a timeout.
(Assignee)

Comment 4

4 years ago
Created attachment 8512344 [details] [diff] [review]
0001-Give-WebGL-conformance-mochis-a-longer-timeout.patch

r=kamidphish
Attachment #8511437 - Attachment is obsolete: true
Attachment #8512344 - Flags: review+
(Assignee)

Updated

4 years ago
Blocks: 1084359
(Assignee)

Updated

4 years ago
Blocks: 1081091
(Assignee)

Updated

4 years ago
Blocks: 1062370
https://hg.mozilla.org/mozilla-central/rev/b78755143fda
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
(Assignee)

Updated

4 years ago
Blocks: 1076799
(Assignee)

Updated

4 years ago
Blocks: 1089415
(Assignee)

Updated

4 years ago
Blocks: 1090337
(Assignee)

Updated

4 years ago
Blocks: 1087226
(Assignee)

Comment 6

4 years ago
Comment on attachment 8512344 [details] [diff] [review]
0001-Give-WebGL-conformance-mochis-a-longer-timeout.patch

Approval Request Comment
[Feature/regressing bug #]: unknown
[User impact if declined]: WebGL 2.3 android tests have high intermittence.
[Describe test coverage new/current, TBPL]: none
[Risks and why]: No user-facing risk.
[String/UUID change made/needed]: none

This is purely to make our tree greener.
Attachment #8512344 - Flags: approval-mozilla-beta?
Attachment #8512344 - Flags: approval-mozilla-aurora?
(Assignee)

Updated

4 years ago
Blocks: 1087302
(Assignee)

Updated

4 years ago
Blocks: 1079619
Comment on attachment 8512344 [details] [diff] [review]
0001-Give-WebGL-conformance-mochis-a-longer-timeout.patch

Test-only changes don't need approval.
Attachment #8512344 - Flags: approval-mozilla-beta?
Attachment #8512344 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/releases/mozilla-aurora/rev/a56669d236e7
https://hg.mozilla.org/releases/mozilla-beta/rev/9a6a63827c10

I'm not convinced this is actually helping, though. I think we may want to split mochitest-gl into 2 chunks as our next option.
status-firefox34: --- → fixed
status-firefox35: --- → fixed
status-firefox36: --- → fixed
(Assignee)

Updated

4 years ago
Blocks: 1088653
(Assignee)

Comment 9

4 years ago
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #8)
> https://hg.mozilla.org/releases/mozilla-aurora/rev/a56669d236e7
> https://hg.mozilla.org/releases/mozilla-beta/rev/9a6a63827c10
> 
> I'm not convinced this is actually helping, though. I think we may want to
> split mochitest-gl into 2 chunks as our next option.

It certainly seems that way.
(Assignee)

Updated

4 years ago
Blocks: 1074581
(Assignee)

Updated

4 years ago
Blocks: 1075458
(Assignee)

Updated

4 years ago
Blocks: 1082482
(Assignee)

Updated

4 years ago
Blocks: 1092201
(In reply to Jeff Gilbert [:jgilbert] from comment #9)
> (In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #8)
> > I'm not convinced this is actually helping, though. I think we may want to
> > split mochitest-gl into 2 chunks as our next option.
> 
> It certainly seems that way.

+1 :)

"application ran for longer than allowed maximum time" means that the test job (mochitest-gl) ran for longer than the maximum time (typically 1 hour). Even when green, Android 2.3 mochitest-gl is running for about an hour now. It is time for 2 chunks.
(Assignee)

Updated

4 years ago
See Also: → bug 1093867
(Assignee)

Comment 11

4 years ago
Ah, ok. Let the chunking begin. That is bug 1093867.
(Assignee)

Updated

4 years ago
Blocks: 1082574
You need to log in before you can comment on or make changes to this bug.