1562167 - Intermittent InvalidArgumentException: Unknown pointerType: [object String] "touch"

Status: RESOLVED FIXED

(Testing :: web-platform-tests, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: hskupin [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=253725183&repo=try
Full log: https://queue.taskcluster.net/v1/task/CmxJWGKXQFedd5N80tPqEA/runs/0/artifacts/public/logs/live_backing.log

---

Separate bug for the intermittent failures in web-platform-tests, so classification and intermittent view doesn't collide with our implementation process on bug 1543337.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Here an example:

https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=253725183&repo=try&lineNumber=4731-4749

[task 2019-06-28T01:42:02.220Z] 01:42:02     INFO - TEST-START | /pointerevents/pointerevent_boundary_events_in_capturing.html
[task 2019-06-28T01:42:02.221Z] 01:42:02     INFO - Closing window 2147483656
[task 2019-06-28T01:42:02.818Z] 01:42:02     INFO - {'actions': [{u'type': u'pointer', u'actions': [{u'y': 0, u'x': 0, u'type': u'pointerMove', u'origin': {'element-6066-11e4-a52e-4f735466cecf': u'd60e51fe-58a9-4d42-a492-41718cabed9a', 'chromeelement-9fc5-4b51-a3c8-01716eedeb04': u'd60e51fe-58a9-4d42-a492-41718cabed9a'}}, {u'button': 0, u'type': u'pointerDown'}, {u'y': 0, u'x': 10, u'type': u'pointerMove', u'origin': {'element-6066-11e4-a52e-4f735466cecf': u'd60e51fe-58a9-4d42-a492-41718cabed9a', 'chromeelement-9fc5-4b51-a3c8-01716eedeb04': u'd60e51fe-58a9-4d42-a492-41718cabed9a'}}, {u'y': 0, u'x': 20, u'type': u'pointerMove', u'origin': {'element-6066-11e4-a52e-4f735466cecf': u'd60e51fe-58a9-4d42-a492-41718cabed9a', 'chromeelement-9fc5-4b51-a3c8-01716eedeb04': u'd60e51fe-58a9-4d42-a492-41718cabed9a'}}, {u'y': 0, u'x': 30, u'type': u'pointerMove', u'origin': {'element-6066-11e4-a52e-4f735466cecf': u'd60e51fe-58a9-4d42-a492-41718cabed9a', 'chromeelement-9fc5-4b51-a3c8-01716eedeb04': u'd60e51fe-58a9-4d42-a492-41718cabed9a'}}, {u'button': 0, u'type': u'pointerUp'}], u'parameters': {u'pointerType': u'mouse'}, u'id': u'mousePointer1'}]}
[task 2019-06-28T01:42:02.946Z] 01:42:02     INFO - {'actions': [{u'type': u'pointer', u'actions': [{u'y': 0, u'x': 0, u'type': u'pointerMove', u'origin': {'element-6066-11e4-a52e-4f735466cecf': u'd60e51fe-58a9-4d42-a492-41718cabed9a', 'chromeelement-9fc5-4b51-a3c8-01716eedeb04': u'd60e51fe-58a9-4d42-a492-41718cabed9a'}}, {u'button': 0, u'type': u'pointerDown'}, {u'y': 0, u'x': 10, u'type': u'pointerMove', u'origin': {'element-6066-11e4-a52e-4f735466cecf': u'd60e51fe-58a9-4d42-a492-41718cabed9a', 'chromeelement-9fc5-4b51-a3c8-01716eedeb04': u'd60e51fe-58a9-4d42-a492-41718cabed9a'}}, {u'y': 0, u'x': 20, u'type': u'pointerMove', u'origin': {'element-6066-11e4-a52e-4f735466cecf': u'd60e51fe-58a9-4d42-a492-41718cabed9a', 'chromeelement-9fc5-4b51-a3c8-01716eedeb04': u'd60e51fe-58a9-4d42-a492-41718cabed9a'}}, {u'y': 0, u'x': 30, u'type': u'pointerMove', u'origin': {'element-6066-11e4-a52e-4f735466cecf': u'd60e51fe-58a9-4d42-a492-41718cabed9a', 'chromeelement-9fc5-4b51-a3c8-01716eedeb04': u'd60e51fe-58a9-4d42-a492-41718cabed9a'}}, {u'button': 0, u'type': u'pointerUp'}], u'parameters': {u'pointerType': u'touch'}, u'id': u'touchPointer1'}]}
[task 2019-06-28T01:42:02.968Z] 01:42:02  WARNING - Action action_sequence failed
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING - Traceback (most recent call last):
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING -   File "/builds/worker/workspace/build/tests/web-platform/tests/tools/wptrunner/wptrunner/executors/base.py", line 618, in process_action
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING -     action_handler(payload)
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING -   File "/builds/worker/workspace/build/tests/web-platform/tests/tools/wptrunner/wptrunner/executors/base.py", line 673, in __call__
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING -     self.protocol.action_sequence.send_actions({"actions": actions})
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING -   File "/builds/worker/workspace/build/tests/web-platform/tests/tools/wptrunner/wptrunner/executors/executormarionette.py", line 413, in send_actions
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING -     self.marionette._send_message("WebDriver:PerformActions", actions)
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING -   File "/builds/worker/workspace/build/venv/lib/python2.7/site-packages/marionette_driver/decorators.py", line 26, in _
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING -     return func(*args, **kwargs)
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING -   File "/builds/worker/workspace/build/venv/lib/python2.7/site-packages/marionette_driver/marionette.py", line 598, in _send_message
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING -     self._handle_error(err)
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING -   File "/builds/worker/workspace/build/venv/lib/python2.7/site-packages/marionette_driver/marionette.py", line 618, in _handle_error
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING -     raise errors.lookup(error)(message, stacktrace=stacktrace)
[task 2019-06-28T01:42:02.969Z] 01:42:02  WARNING - InvalidArgumentException: Unknown pointerType: [object String] "touch"

This seems to come from the CallbackHandler class:

https://searchfox.org/mozilla-central/rev/f91bd38732d4a330eba4e780812274b98eb81274/testing/web-platform/tests/tools/wptrunner/wptrunner/executors/base.py#586

So tests might call into here with those actions. I wonder if we should skip those until we really support the touch action primitive?

Comment 2

[David Burns :automatedtester]

Those tests should have expectation data showing them as failing until we fix this. We should not skip them

Comment 3

[Intermittent Failures Robot]

58 failures in 815 pushes (0.071 failures/push) were associated with this bug yesterday.

Repository breakdown:

• autoland: 21
• mozilla-central: 8
• try: 9
• mozilla-beta: 2
• mozilla-inbound: 18

Platform breakdown:

• android-em-7-0-x86_64: 46
• macosx1014-64-shippable: 1
• linux64-qr: 3
• linux64-shippable-qr: 1
• linux64-ccov: 1
• linux32-shippable: 1
• linux64-shippable: 1
• linux64: 2
• macosx1014-64: 1
• macosx1010-64-shippable: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1562167&startday=2019-06-28&endday=2019-06-28&tree=all

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+2]]

(In reply to David Burns :automatedtester from comment #2)

Those tests should have expectation data showing them as failing until we fix this. We should not skip them

The problem is that it is only a warning and pretty much fills the failure pane on Treeherder without failing the tests. So marking as skipped or expected fail won't work in both cases.

Comment 5

[Intermittent Failures Robot]

104 failures in 4409 pushes (0.024 failures/push) were associated with this bug in the last 7 days.

This is the #16 most frequent failure this week.

** This failure happened more than 75 times this week! Resolving this bug is a very high priority. **

** Try to resolve this bug as soon as possible. If unresolved for 1 week, the affected test(s) may be disabled. **

Repository breakdown:

• try: 15
• autoland: 47
• mozilla-central: 15
• mozilla-beta: 3
• mozilla-inbound: 24

Platform breakdown:

• android-em-7-0-x86_64: 85
• macosx1014-64-shippable: 1
• linux64-qr: 5
• linux64-shippable-qr: 1
• linux64-ccov: 1
• linux64-shippable: 1
• linux32-shippable: 2
• linux64: 4
• macosx1014-64: 2
• macosx1010-64-shippable: 1
• linux32: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1562167&startday=2019-06-24&endday=2019-06-30&tree=all

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+2]]

David, could you maybe help with it while James is away? Otherwise I may have to change priorities.

Comment 7

[Intermittent Failures Robot]

32 failures in 509 pushes (0.063 failures/push) were associated with this bug yesterday.

Repository breakdown:

• autoland: 22
• mozilla-beta: 2
• mozilla-central: 7
• mozilla-inbound: 1

Platform breakdown:

• android-em-7-0-x86_64: 24
• linux64: 2
• linux64-qr: 1
• linux64-ccov: 1
• windows10-64-ccov: 1
• macosx1014-64: 3

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1562167&startday=2019-07-01&endday=2019-07-01&tree=all

Comment 8

[David Burns :automatedtester]

Maja, Is this on your list of bugs you are looking for disabling? If not could you disable it, I am already doing the marionette changes for this

Comment 9

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

This doesn't actually cause jobs to be orange so I don't think it's a good idea to disable just for warning logs, especially given that we already have accurate expectation metadata. If I'm missing something, let me know.

Comment 10

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Now that the intermittent-failure keyword has been removed, we shouldn't get new classifications anymore. Maybe that helps for the time being.

Comment 11

[Intermittent Failures Robot]

38 failures in 3501 pushes (0.011 failures/push) were associated with this bug in the last 7 days.

** This failure happened more than 30 times this week! Resolving this bug is a high priority. **

** Try to resolve this bug as soon as possible. If unresolved for 2 weeks, the affected test(s) may be disabled. **

Repository breakdown:

• autoland: 23
• mozilla-beta: 2
• mozilla-central: 7
• try: 1
• mozilla-inbound: 5

Platform breakdown:

• android-em-7-0-x86_64: 29
• linux64: 3
• linux64-qr: 1
• linux64-ccov: 1
• windows10-64-ccov: 1
• macosx1014-64: 3

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1562167&startday=2019-07-01&endday=2019-07-07&tree=all

Comment 12

[Firefox Bug Husbandry Bot]

https://wiki.mozilla.org/Bug_Triage#Intermittent_Test_Failure_Cleanup

Comment 13

[Henrik Skupin [:whimboo][⌚️UTC+2]]

This is not fixed but no longer marked as intermittent failure.

Comment 14

[Intermittent Failures Robot]

1 failures in 4314 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• mozilla-central: 1

Platform breakdown:

• linux64-shippable-qr: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1562167&startday=2019-10-28&endday=2019-11-03&tree=all

Comment 15

[BugBot [:suhaib / :marco/ :calixte]]

https://wiki.mozilla.org/Bug_Triage#Intermittent_Test_Failure_Cleanup
For more information, please visit auto_nag documentation.

Comment 16

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Adding leave-open to ensure it stays open until the dependency has been fixed.

Comment 17

[Intermittent Failures Robot]

7 failures in 3773 pushes (0.002 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• autoland: 4
• try: 3

Platform breakdown:

• linux1804-64-qr: 3
• linux64-shippable: 1
• linux1804-64-shippable-qr: 1
• linux1804-32-shippable: 1
• macosx1014-64-shippable: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1562167&startday=2020-01-13&endday=2020-01-19&tree=all

Comment 18

[Intermittent Failures Robot]

1 failures in 2190 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• try: 1

Platform breakdown:

• linux1804-64-shippable-qr: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1562167&startday=2020-01-27&endday=2020-02-02&tree=all

Comment 19

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

Attachment: Bug 1562167 - Avoid log-parser highlight for marionette pointerType error

If we only log the error without raising, hundreds of tests would report
a timeout in Mozilla's CI.

The Treeherder log highlighting excludes strings that include "TEST-INFO ".

Comment 20

[BugBot [:suhaib / :marco/ :calixte]]

The leave-open keyword is there and there is no activity for 6 months.
:maja_zf, maybe it's time to close this bug?

Comment 21

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

The attached patch fixes the problem, but extensive metadata and test changes need to be added. I'm keeping the leave-open and hopefully I can revive that patch soon.

Comment 22

[Intermittent Failures Robot]

1 failures in 3810 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• mozilla-esr78: 1

Platform breakdown:

• windows7-32-shippable: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1562167&startday=2021-01-11&endday=2021-01-17&tree=all

Comment 23

[BugBot [:suhaib / :marco/ :calixte]]

The leave-open keyword is there and there is no activity for 6 months.
:jgraham, maybe it's time to close this bug?

Comment 24

[Henrik Skupin [:whimboo][⌚️UTC+2]]

It's still needed, but maybe it should be easy to finish by taking Maja's patch for the remaining work?

Comment 25

[BugBot [:suhaib / :marco/ :calixte]]

The leave-open keyword is there and there is no activity for 6 months.
:jgraham, maybe it's time to close this bug?

Comment 26

[Henrik Skupin [:whimboo][⌚️UTC+2]]

James, I assume this bug can now be marked as fixed?

Comment 27

[James Graham [:jgraham]]

This was fixed as part of 1543337

Comment 28

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Great. So lets also mark it as fixed given that we know the bug.