WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
63793
new-run-webkit-tests output gets confused when logging
https://bugs.webkit.org/show_bug.cgi?id=63793
Summary
new-run-webkit-tests output gets confused when logging
Adam Barth
Reported
2011-07-01 01:10:56 PDT
new-run-webkit-tests output gets confused when logging
Attachments
Patch
(2.09 KB, patch)
2011-07-01 01:13 PDT
,
Adam Barth
no flags
Details
Formatted Diff
Diff
Patch
(9.42 KB, patch)
2012-06-14 22:31 PDT
,
Dirk Pranke
no flags
Details
Formatted Diff
Diff
Patch
(10.88 KB, patch)
2012-06-15 14:48 PDT
,
Dirk Pranke
abarth
: review+
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Adam Barth
Comment 1
2011-07-01 01:13:09 PDT
Created
attachment 99443
[details]
Patch
Eric Seidel (no email)
Comment 2
2011-07-01 01:17:31 PDT
Comment on
attachment 99443
[details]
Patch OK. How do we test this?
Adam Barth
Comment 3
2011-07-01 01:31:00 PDT
> OK. How do we test this?
Dunno. It actually fails in a different way. :(
Dirk Pranke
Comment 4
2011-07-01 15:59:59 PDT
There are actually unit tests for this module (metered_stream_unittest.py). You can at least update them with what the expected behavior would be. I'm not surprised if it is failing in a different way, because I remember having to play with the ordering once or twice to get it to even mostly work. One solution would be to route all logging from the workers to the master so that it could coordinate the cursor state. I looked into doing that once (when we were getting duplicate messages in the log from the workers) but only got about halfway through before I realized the problem was actually that there were two loggers running in that case. Another (presumably much less portable) option would be to push the MeteredStream's state into some form of shared memory.
Adam Barth
Comment 5
2011-07-01 16:30:24 PDT
Comment on
attachment 99443
[details]
Patch Logging in NRWT is way too complicated. I'd prefer to just do something simpler.
Dirk Pranke
Comment 6
2012-06-08 18:59:18 PDT
***
Bug 67164
has been marked as a duplicate of this bug. ***
Dirk Pranke
Comment 7
2012-06-14 22:31:48 PDT
Created
attachment 147733
[details]
Patch
Dirk Pranke
Comment 8
2012-06-15 14:48:51 PDT
Created
attachment 147902
[details]
Patch
Adam Barth
Comment 9
2012-06-18 11:52:21 PDT
Comment on
attachment 147902
[details]
Patch I don't really understand python logging enough to really understanding this patch. Are you looking for a real review or just a rubber stamp?
Adam Barth
Comment 10
2012-06-18 12:10:43 PDT
Comment on
attachment 147902
[details]
Patch dpranke told me in #webkit that he's just look for an rs here, which I'm happy to provide.
Dirk Pranke
Comment 11
2012-06-18 13:00:07 PDT
Committed
r120619
: <
http://trac.webkit.org/changeset/120619
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug