Closed Bug 1361325 Opened 7 years ago Closed 5 years ago

Autophone Mdm4 perma fail on Android 5.1/Nexus 6

Categories

(Core :: WebRTC, defect, P3)

defect

Tracking

()

RESOLVED FIXED
Tracking Status
fennec + ---

People

(Reporter: bc, Unassigned)

References

Details

(Whiteboard: [stockwell disabled])

Attachments

(2 files)

Attached file failures
The following tests in Mdm4 fail on Android 5.1/Nexus 6

dom/media/tests/mochitest/identity/test_fingerprints.html
dom/media/tests/mochitest/identity/test_getIdentityAssertion.html
dom/media/tests/mochitest/identity/test_idpproxy.html
dom/media/tests/mochitest/identity/test_idpproxy.html
dom/media/tests/mochitest/identity/test_idpproxy.html
dom/media/tests/mochitest/identity/test_idpproxy.html
dom/media/tests/mochitest/identity/test_idpproxy.html
dom/media/tests/mochitest/identity/test_loginNeeded.html
dom/media/tests/mochitest/identity/test_loginNeeded.html:32:13
dom/media/tests/mochitest/identity/test_peerConnection_asymmetricIsolation.html
dom/media/tests/mochitest/identity/test_peerConnection_peerIdentity.html
dom/media/tests/mochitest/identity/test_setIdentityProvider.html
dom/media/tests/mochitest/identity/test_setIdentityProviderWithErrors.html


https://autophone.s3.amazonaws.com/v1/task/ByjGz98ZR5u342zVyaIDsA/runs/0/artifacts/public/build/mochitest-dom-media-tests-identity-mochitests-dom-media-tests-identity-settings-debug.ini-1-nexus-6-1-a9927acd-1dc2-4123-a6c0-ffd9d497cdac.log

These will be disabled in bug 1358876.
Any idea what is going on with these tests? We had them disabled in the old Mw job, but they run fine locally and I had a green try push with them enabled back in April (https://treeherder.mozilla.org/#/jobs?repo=try&revision=c49df13eb73f7774f64063dc078c80ef4de0a49e).
No, I don't but that is certainly very interesting.
Blake, will media team triage this?
Flags: needinfo?(bwu)
We (TPE media team) don't triage this. It looks like related to WebRTC's signaling part.
Flags: needinfo?(bwu)
tracking-fennec: ? → +
Martin, do you have any idea what could cause these test failure messages:

Error in test execution: IdpError: Load failed: 2153398266
Rank: 25
Flags: needinfo?(martin.thomson)
Priority: -- → P2
I'd be more concerned about this:

5 INFO TEST-UNEXPECTED-FAIL | dom/media/tests/mochitest/identity/test_fingerprints.html | error in test: Promise rejection value is a non-unwrappable cross-compartment wrapper.

That message means that the real error is being hidden.  We'd want to add some logging to determine what that is.

The number appears to be an nsresult value, but I can't find its definition anywhere.
Flags: needinfo?(martin.thomson)
The Mdm4 tests are completely disabled in the in-tree manifests for Android 5.1 which causes them to be perma orange on mozilla-central. This patch changes the autophone manifests so the test is no longer scheduled for Android 5.1 on mozilla-central. It will still be available on try.
Attachment #8865864 - Flags: review?(jmaher)
Comment on attachment 8865864 [details] [diff] [review]
bug-1361325-autophone-disable-Mdm4-nexus-6.patch

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

lgtm
Attachment #8865864 - Flags: review?(jmaher) → review+
can we mark this as closed?
Whiteboard: [stockwell needswork] → [stockwell disabled]
(In reply to Joel Maher ( :jmaher) from comment #13)
> can we mark this as closed?

I think we should leave this open as it appears there is a real issue here that requires investigation, as per comment 7.
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:bc, could you have a look please?

Flags: needinfo?(bob)

This patch did land but in the autophone github repo. Autophone no longer exists and we do not run the mdm4 tests on android-hw. I'll defer to dminor as to the resolution considering his comment 14.

Flags: needinfo?(bob) → needinfo?(dminor)

I think we can mark this as FIXED (since the patch landed on autophone) and open a new bug in the future if necessary.

Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(dminor)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: