Closed
Bug 1046566
Opened 11 years ago
Closed 11 years ago
Almost every line is printed twice in B2G Desktop Linux Opt Full Log
Categories
(Testing :: Mochitest, defect)
Testing
Mochitest
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla34
People
(Reporter: TYLin, Assigned: akachkach)
References
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
2.08 KB,
patch
|
akachkach
:
review+
|
Details | Diff | Splinter Review |
See the BAD log. For example:
16:13:36 INFO - 43346 INFO TEST-START | Shutdown
16:13:36 INFO - 43347 INFO TEST-START | Shutdown
16:13:36 INFO - 43348 INFO Passed: 391711
16:13:36 INFO - 43349 INFO Passed: 391711
16:13:36 INFO - 43350 INFO Failed: 0
16:13:36 INFO - 43351 INFO Failed: 0
16:13:36 INFO - 43352 INFO Todo: 8737
16:13:36 INFO - 43353 INFO Todo: 8737
16:13:36 INFO - 43354 INFO Slowest: 36175ms - /tests/image/test/mochitest/test_bug733553.html
16:13:36 INFO - 43355 INFO Slowest: 36175ms - /tests/image/test/mochitest/test_bug733553.html
16:13:36 INFO - 43356 INFO SimpleTest FINISHED
16:13:36 INFO - 43357 INFO SimpleTest FINISHED
16:13:36 INFO - 43358 INFO TEST-INFO | Ran 1 Loops
16:13:36 INFO - 43359 INFO TEST-INFO | Ran 1 Loops
16:13:36 INFO - 43360 INFO SimpleTest FINISHED
16:13:36 INFO - 43361 INFO SimpleTest FINISHED
16:13:36 INFO - 43362 INFO SUITE-END | took 1097s
16:13:36 INFO - 43363 INFO SUITE-END | took 1097s
BAD @ 2014-07-30 15:53:38 PDT
https://tbpl.mozilla.org/php/getParsedLog.php?id=44928000&tree=Mozilla-Central&full=1
Good @ 2014-07-30 15:09:49 PDT
https://tbpl.mozilla.org/php/getParsedLog.php?id=44924888&tree=Mozilla-Central&full=1
Comment 1•11 years ago
|
||
Thank you for filing :-)
TBPL only displays the results and logs generated elsewhere - moving to mochitest.
Comment 2•11 years ago
|
||
Good:
https://tbpl.mozilla.org/php/getParsedLog.php?id=44896337&full=1&branch=mozilla-inbound
Bad:
https://tbpl.mozilla.org/php/getParsedLog.php?id=44895449&full=1&branch=mozilla-inbound
Caused by:
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=327f8728d818
Bug 1044206 - Revert the global logger in mochitest's runtests to use python stdlib logging for compatibility with mozharness regex.;r=ahal
Blocks: 1044206
Assignee | ||
Comment 3•11 years ago
|
||
Looks like we're attaching two handlers to the same logger? Still, this is pretty unlikely because the StructuredLogger in that patch is instantiated directly before we attach the StreamHandler.
Are messages passed to the handlers of loggers that have the same name (even if it's two different instances)?
Flags: needinfo?(james)
Comment 4•11 years ago
|
||
Yes, all loggers with the same name share the same handlers.
Flags: needinfo?(james)
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → akachkach
Assignee | ||
Comment 5•11 years ago
|
||
A quick fix for this would be to make sure we don't instantiate multiple structured loggers.
Just did a try push to see if it's enough: https://tbpl.mozilla.org/?tree=Try&rev=1bee4ba52f4d
Attachment #8465979 -
Flags: review?(ahalberstadt)
Comment 6•11 years ago
|
||
Comment on attachment 8465979 [details] [diff] [review]
0001-Bug-1046566-avoid-instantiating-multiple-structured-.patch
Review of attachment 8465979 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks, probably the easiest solution for now.
Attachment #8465979 -
Flags: review?(ahalberstadt) → review+
Reporter | ||
Comment 7•11 years ago
|
||
I notice that B2G Desktop Linux Opt had an exception.
UnboundLocalError: local variable 'structured_log' referenced before assignment
Assignee | ||
Comment 8•11 years ago
|
||
carry r+ forward
Thanks TYLin, fixed that bug and looks like it eliminates the duplicated output.
https://tbpl.mozilla.org/?rev=b2186e7af31b&tree=Try
Attachment #8465979 -
Attachment is obsolete: true
Attachment #8466376 -
Flags: review+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 9•11 years ago
|
||
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Comment 10•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → mozilla34
You need to log in
before you can comment on or make changes to this bug.
Description
•