Bug 108427

Summary: EWS graphs have unusual data for patch waiting times
Product: WebKit Reporter: Alan Cutter <alancutter>
Component: Tools / TestsAssignee: Alan Cutter <alancutter>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

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.