Bug 108427 - EWS graphs have unusual data for patch waiting times
Summary: EWS graphs have unusual data for patch waiting times
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alan Cutter
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-30 22:27 PST by Alan Cutter
Modified: 2013-01-31 11:36 PST (History)
3 users (show)

See Also:


Attachments
Patch (16.58 KB, patch)
2013-01-31 04:51 PST, Alan Cutter
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alan Cutter 2013-01-30 22:27:45 PST
http://i.imgur.com/i8lcG92.png
http://i.imgur.com/jyUHdxq.png
http://i.imgur.com/sBwdHEA.png

There are giant spikes on some patch waiting time graphs and linearly increasing data on others.
It is not uncommon to see patch waiting times logged on the order of days.
Single patch waiting time scatter plots don't always match up with the waiting times shown for the queue where particularly noticeable.
Comment 1 Alan Cutter 2013-01-31 04:51:39 PST
Created attachment 185751 [details]
Patch
Comment 2 WebKit Review Bot 2013-01-31 04:58:47 PST
Attachment 185751 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Tools/ChangeLog', u'Tools/QueueStatusServer/app.yaml', u'Tools/QueueStatusServer/cron.yaml', u'Tools/QueueStatusServer/handlers/syncqueuelogs.py', u'Tools/QueueStatusServer/loggers/recordpatchevent.py', u'Tools/QueueStatusServer/main.py', u'Tools/QueueStatusServer/model/patchlog.py', u'Tools/QueueStatusServer/model/queuelog.py', u'Tools/QueueStatusServer/model/warninglog.py']" exit_code: 1
Tools/QueueStatusServer/model/warninglog.py:32:  No name 'appengine' in module 'google'  [pylint/E0611] [5]
Tools/QueueStatusServer/model/warninglog.py:46:  [WarningLog.record] Instance of 'WarningLog' has no 'put' member  [pylint/E1101] [5]
Tools/QueueStatusServer/handlers/syncqueuelogs.py:29:  No name 'appengine' in module 'google'  [pylint/E0611] [5]
Tools/QueueStatusServer/handlers/syncqueuelogs.py:42:  [SyncQueueLogs.get] Instance of 'SyncQueueLogs' has no 'response' member  [pylint/E1101] [5]
Tools/QueueStatusServer/model/patchlog.py:53:  [PatchLog.lookup_if_exists] Class 'PatchLog' has no 'get_by_key_name' member  [pylint/E1101] [5]
Total errors found: 5 in 9 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Alan Cutter 2013-01-31 05:08:28 PST
(In reply to comment #2)
> Attachment 185751 [details] did not pass style-queue:
> 
> Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Tools/ChangeLog', u'Tools/QueueStatusServer/app.yaml', u'Tools/QueueStatusServer/cron.yaml', u'Tools/QueueStatusServer/handlers/syncqueuelogs.py', u'Tools/QueueStatusServer/loggers/recordpatchevent.py', u'Tools/QueueStatusServer/main.py', u'Tools/QueueStatusServer/model/patchlog.py', u'Tools/QueueStatusServer/model/queuelog.py', u'Tools/QueueStatusServer/model/warninglog.py']" exit_code: 1
> Tools/QueueStatusServer/model/warninglog.py:32:  No name 'appengine' in module 'google'  [pylint/E0611] [5]
> Tools/QueueStatusServer/model/warninglog.py:46:  [WarningLog.record] Instance of 'WarningLog' has no 'put' member  [pylint/E1101] [5]
> Tools/QueueStatusServer/handlers/syncqueuelogs.py:29:  No name 'appengine' in module 'google'  [pylint/E0611] [5]
> Tools/QueueStatusServer/handlers/syncqueuelogs.py:42:  [SyncQueueLogs.get] Instance of 'SyncQueueLogs' has no 'response' member  [pylint/E1101] [5]
> Tools/QueueStatusServer/model/patchlog.py:53:  [PatchLog.lookup_if_exists] Class 'PatchLog' has no 'get_by_key_name' member  [pylint/E1101] [5]
> Total errors found: 5 in 9 files
> 
> 
> If any of these errors are false positives, please file a bug against check-webkit-style.

Created bug 108463 to address this issue.
Comment 4 Eric Seidel (no email) 2013-01-31 11:15:27 PST
Comment on attachment 185751 [details]
Patch

LGTM.
Comment 5 WebKit Review Bot 2013-01-31 11:36:11 PST
Comment on attachment 185751 [details]
Patch

Clearing flags on attachment: 185751

Committed r141451: <http://trac.webkit.org/changeset/141451>
Comment 6 WebKit Review Bot 2013-01-31 11:36:15 PST
All reviewed patches have been landed.  Closing bug.