"TypeError: on_timeout() takes exactly 2 arguments (1 given)" in mozprocess/processhandler.py", line 724, in onTimeout

RESOLVED FIXED in mozilla33

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: emorley, Assigned: ahal)

Tracking

Trunk
mozilla33
ARM
Gonk (Firefox OS)
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

4 years ago
Broken out from bug 1027960.

Andrew, I think this is fallout from bug 997244, given the timing of the filing of bug 1027960 and that this has only been seen on B2G so far. Would you mind taking a look? :-)
Flags: needinfo?(ahalberstadt)
(Reporter)

Comment 1

4 years ago
Sorry, the log:

https://tbpl.mozilla.org/php/getParsedLog.php?id=42134471&tree=B2g-Inbound
b2g_emulator_vm b2g-inbound opt test mochitest-4 on 2014-06-20 03:03:23 PDT for push 49f37e156352

{
04:01:22     INFO -  1148 INFO TEST-START | /tests/docshell/test/test_bug529119-1.html
04:01:24     INFO -  ###################################### forms.js loaded
04:01:24     INFO -  ############################### browserElementPanning.js loaded
04:01:24     INFO -  ######################## BrowserElementChildPreload.js loaded
04:01:27     INFO -  ############ ErrorPage.js
04:01:27     INFO -  ############ ErrorPage.js
04:01:29     INFO -  1149 INFO TEST-INFO | MEMORY STAT vsize after test: 131358720
04:01:29     INFO -  1150 INFO TEST-INFO | MEMORY STAT residentFast after test: 54571008
04:01:29     INFO -  1151 INFO TEST-INFO | MEMORY STAT heapAllocated after test: 27180984
04:07:00     INFO -  Exception in thread Thread-6:
04:07:00     INFO -  Traceback (most recent call last):
04:07:00     INFO -    File "/usr/lib/python2.7/threading.py", line 551, in __bootstrap_inner
04:07:00     INFO -      self.run()
04:07:00     INFO -    File "/usr/lib/python2.7/threading.py", line 504, in run
04:07:00     INFO -      self.__target(*self.__args, **self.__kwargs)
04:07:00     INFO -    File "/builds/slave/test/build/venv/local/lib/python2.7/site-packages/mozprocess/processhandler.py", line 786, in _processOutput
04:07:00     INFO -      self.onTimeout()
04:07:00     INFO -    File "/builds/slave/test/build/venv/local/lib/python2.7/site-packages/mozprocess/processhandler.py", line 724, in onTimeout
04:07:00     INFO -      handler()
04:07:00     INFO -  TypeError: on_timeout() takes exactly 2 arguments (1 given)
04:07:00     INFO -  Stopping web server
04:08:03     INFO -  Stopping web socket server
04:08:03     INFO -  Stopping ssltunnel
04:08:03     INFO -  Mochitest INFO | runtestsb2g.py | Running tests: end.
}
(Assignee)

Updated

4 years ago
Assignee: nobody → ahalberstadt
Flags: needinfo?(ahalberstadt)
(Assignee)

Comment 2

4 years ago
Created attachment 8443444 [details] [diff] [review]
fix error on timeout

Bah, stupid mistake. It must have either not shown up in try, or I missed it when it did. Thanks for catching this!
Attachment #8443444 - Flags: review?(emorley)
(Assignee)

Comment 3

4 years ago
Comment on attachment 8443444 [details] [diff] [review]
fix error on timeout

Hang on, there's another error in there that this will expose..
Attachment #8443444 - Attachment is obsolete: true
Attachment #8443444 - Flags: review?(emorley)
(Assignee)

Comment 4

4 years ago
Created attachment 8443451 [details] [diff] [review]
fix error on timeout

This fixes a few other errors I noticed. Note this whole function should get moved somewhere else as part of bug 1025051. I may also need to get Ahmed to update his structured logging patch to do this in a structured way.
Attachment #8443451 - Flags: review?(emorley)
(Reporter)

Comment 5

4 years ago
Comment on attachment 8443451 [details] [diff] [review]
fix error on timeout

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

lgtm; thank you for the quick turnaround :-)
Attachment #8443451 - Flags: review?(emorley) → review+
(Assignee)

Updated

4 years ago
Blocks: 1028254
(Assignee)

Comment 8

4 years ago
Created attachment 8443597 [details] [diff] [review]
change timeout string

This should hide the perma-orange in m-9, but not fix it. We still need to investigate why this is happening, and why it wasn't previously turning orange. There is a chance that b2g emulator timeouts haven't been working properly for awhile now.
Attachment #8443597 - Flags: review?(emorley)
(Reporter)

Comment 9

4 years ago
Comment on attachment 8443597 [details] [diff] [review]
change timeout string

I'm only partially sure we may trick the parser this way - it may be that the refactor fixed something else too. Worth a try though (failing this, I guess we just hide debug m-9 until the shutdown timeout is fixed).
Attachment #8443597 - Flags: review?(emorley) → review+
(Assignee)

Comment 10

4 years ago
Yeah, that could be. I guess we'll find out soon:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8965c073da53
(Assignee)

Updated

4 years ago
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/4cffdbeec391
https://hg.mozilla.org/mozilla-central/rev/8965c073da53
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in before you can comment on or make changes to this bug.