Closed
Bug 809447
Opened 12 years ago
Closed 12 years ago
mozharness should use ERROR not INFO for "Could not successfully complete transport of message to Gecko, socket closed?"
Categories
(Release Engineering :: Applications: MozharnessCore, defect)
Release Engineering
Applications: MozharnessCore
Other
Gonk (Firefox OS)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: mozilla)
References
(Blocks 1 open bug)
Details
(Whiteboard: [mozharness][sheriff-want])
Attachments
(1 file)
2.28 KB,
patch
|
jgriffin
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
eg:
https://tbpl.mozilla.org/php/getParsedLog.php?id=16822692&tree=Mozilla-Inbound
{
07:00:59 INFO - restarting B2G
07:00:59 INFO - Traceback (most recent call last):
07:00:59 INFO - File "runreftestb2g.py", line 563, in <module>
07:00:59 INFO - sys.exit(main())
07:00:59 INFO - File "runreftestb2g.py", line 483, in main
07:00:59 INFO - marionette = Marionette(**kwargs)
07:00:59 INFO - File "/home/cltbld/talos-slave/test/build/venv/lib/python2.6/site-packages/marionette/marionette.py", line 148, in __init__
07:00:59 INFO - self.emulator.install_gecko(self.gecko_path, self)
07:00:59 INFO - File "/home/cltbld/talos-slave/test/build/venv/lib/python2.6/site-packages/marionette/emulator.py", line 407, in install_gecko
07:00:59 INFO - self.wait_for_system_message(marionette)
07:00:59 INFO - File "/home/cltbld/talos-slave/test/build/venv/lib/python2.6/site-packages/marionette/emulator.py", line 285, in wait_for_system_message
07:00:59 INFO - marionette.start_session()
07:00:59 INFO - File "/home/cltbld/talos-slave/test/build/venv/lib/python2.6/site-packages/marionette/marionette.py", line 272, in start_session
07:00:59 INFO - self.session = self._send_message('newSession', 'value')
07:00:59 INFO - File "/home/cltbld/talos-slave/test/build/venv/lib/python2.6/site-packages/marionette/marionette.py", line 169, in _send_message
07:00:59 INFO - response = self.client.send(message)
07:00:59 INFO - File "/home/cltbld/talos-slave/test/build/venv/lib/python2.6/site-packages/marionette/client.py", line 79, in send
07:00:59 INFO - self.connect()
07:00:59 INFO - File "/home/cltbld/talos-slave/test/build/venv/lib/python2.6/site-packages/marionette/client.py", line 67, in connect
07:00:59 INFO - hello = self.receive()
07:00:59 INFO - File "/home/cltbld/talos-slave/test/build/venv/lib/python2.6/site-packages/marionette/client.py", line 52, in receive
07:00:59 INFO - raise MarionetteException("Could not successfully complete transport of message to Gecko, socket closed?")
07:00:59 INFO - marionette.errors.MarionetteException/home/cltbld/talos-slave/test/build/venv/lib/python2.6/site-packages/marionette/errors.py:18: DeprecationWarning: BaseException.message has been deprecated as of Python 2.6
07:00:59 INFO - return str(self.message)
07:00:59 INFO - : Could not successfully complete transport of message to Gecko, socket closed?
07:00:59 ERROR - Return code: 1
07:00:59 INFO - TinderboxPrint: reftest<br/><em class="testfail">T-FAIL</em>
}
Assignee | ||
Comment 1•12 years ago
|
||
This would be fixed by PythonErrorList (detecting tracebacks) + context lines (bug 688338) but it's easy enough to add this specific error check.
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #679186 -
Flags: review?(jgriffin)
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → aki
Comment 3•12 years ago
|
||
Comment on attachment 679186 [details] [diff] [review]
add requested error check
Thanks!
Attachment #679186 -
Flags: review?(jgriffin) → review+
Assignee | ||
Comment 4•12 years ago
|
||
Comment on attachment 679186 [details] [diff] [review]
add requested error check
http://hg.mozilla.org/build/mozharness/rev/739cfaf3d165
Attachment #679186 -
Flags: checked-in+
Assignee | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 5•12 years ago
|
||
Thank you :-D
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
Updated•11 years ago
|
Component: Other → Mozharness
You need to log in
before you can comment on or make changes to this bug.
Description
•