Intermittent TEST-UNEXPECTED-TIMEOUT | /_mozilla/service-workers/service-worker/clients-matchall-include-uncontrolled.https.html | Verify matchAll() respect includeUncontrolled - Test timed out

RESOLVED FIXED in Firefox 45

Status

Testing
web-platform-tests
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: philor, Assigned: Ehsan)

Tracking

({intermittent-failure})

Trunk
mozilla46
intermittent-failure
Points:
---

Firefox Tracking Flags

(firefox45 fixed, firefox46 fixed)

Details

Attachments

(2 attachments)

Comment 1

2 years ago
19 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 8
* fx-team: 6
* mozilla-central: 5

Platform breakdown:
* osx-10-10: 19

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1230042&startday=2015-12-04&endday=2015-12-04&tree=all

Comment 2

2 years ago
38 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 17
* fx-team: 13
* mozilla-central: 8

Platform breakdown:
* osx-10-10: 38

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1230042&startday=2015-11-30&endday=2015-12-06&tree=all
bkelly: this seems to be a relatively common orange. Might just need a long timeout or something, but you seem like the right person to look at it.

Comment 4

2 years ago
Error message:

  TEST-UNEXPECTED-ERROR | /_mozilla/service-workers/service-worker/clients-matchall-include-uncontrolled.https.html | Error: assert_equals: expected 4 but got 5

I wonder if there is an extra window from a previous tests that is being picked up.  It could be racing with GC to remove the window or something.
Assignee: nobody → bkelly
Blocks: 1173500
Status: NEW → ASSIGNED

Comment 5

2 years ago
17 automation job failures were associated with this bug yesterday.

Repository breakdown:
* mozilla-inbound: 7
* try: 4
* fx-team: 4
* mozilla-central: 2

Platform breakdown:
* osx-10-10: 13
* windows7-32: 4

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1230042&startday=2015-12-10&endday=2015-12-10&tree=all

Comment 6

2 years ago
Sucks that this mainly triggers on osx 10.10 which doesn't seem to run in try builds.
(Reporter)

Comment 7

2 years ago
It does, you just have to beg for it. try: -b o -p macosx64 -u web-platform-tests-4[10.10] -t none (or everything from checking every checkbox in the "Restrict tests to platform(s)" section of trychooser if you want all platforms including 10.10)
(Reporter)

Comment 8

2 years ago
Or in theory you could add them to an existing try push that has a completed Mac build, with either the treeherder UI from the downarrow menu at the top right of the push, "Add new Jobs", or with http://try-extender.herokuapp.com/, though neither seems to actually work, or with https://mozilla-ci-tools.readthedocs.org/en/latest/scripts.html which will work but requires a fair amount of effort.

Comment 9

2 years ago
Ehsan asked me to disable this test so he can investigate it next week.
Assignee: bkelly → ehsan

Comment 10

2 years ago
Created attachment 8697595 [details] [diff] [review]
Disable clients-matchall-include-uncontrolled.https.html due to instabilitiy. r=ehsan

Ehsan gave me r+ in person for this.
Attachment #8697595 - Flags: review+

Updated

2 years ago
Keywords: leave-open

Comment 13

2 years ago
57 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 23
* mozilla-central: 14
* fx-team: 12
* try: 8

Platform breakdown:
* osx-10-10: 49
* windows7-32: 8

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1230042&startday=2015-12-07&endday=2015-12-13&tree=all
(Assignee)

Comment 14

2 years ago
Created attachment 8698447 [details] [diff] [review]
Fix clients-matchall-include-uncontrolled.https.html by sorting the clients array and ignoring non-test clients

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9380b7a2b375
Attachment #8698447 - Flags: review?(bkelly)

Updated

2 years ago
Attachment #8698447 - Flags: review?(bkelly) → review+
(Assignee)

Updated

2 years ago
Keywords: leave-open

Comment 16

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6d57f8b28334
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox46: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46

Comment 17

2 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/d044c3abe019
status-firefox45: affected → fixed
You need to log in before you can comment on or make changes to this bug.