Bug 80016

Summary: Perf-o-matic: generate dashboard images for 30, 90, and 365 days
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: Tools / TestsAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, morrita
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 77037    
Attachments:
Description Flags
Patch
none
Removed logging in RunsChartHandler since it'll be too long for 90/365 days eric: review+

Ryosuke Niwa
Reported 2012-03-01 01:40:25 PST
Right now perf-o-matic only generates images for 7 days, and changing the displayDays results in an error. We should generate 30, 90, and 365 days graphs occasionally as well.
Attachments
Patch (34.32 KB, patch)
2012-03-01 01:51 PST, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
Removed logging in RunsChartHandler since it'll be too long for 90/365 days (34.04 KB, patch)
2012-03-01 01:54 PST, Ryosuke Niwa 		eric: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2012-03-01 01:51:51 PST
Created attachment 129665 [details] Patch
Ryosuke Niwa
Comment 2 2012-03-01 01:54:08 PST
Created attachment 129667 [details] Removed logging in RunsChartHandler since it'll be too long for 90/365 days
Ryosuke Niwa
Comment 3 2012-03-01 01:56:28 PST
Comment on attachment 129667 [details] Removed logging in RunsChartHandler since it'll be too long for 90/365 days View in context: https://bugs.webkit.org/attachment.cgi?id=129667&action=review > Websites/webkit-perf.appspot.com/controller.py:-171 > - self.response.out.write('Fetched http://chart.googleapis.com/chart?%s' % urllib.urlencode(params)) > - Note: this will be too big to print for 90 & 365 days.
Eric Seidel (no email)
Comment 4 2012-03-01 13:50:40 PST
Comment on attachment 129667 [details] Removed logging in RunsChartHandler since it'll be too long for 90/365 days OK.
Ryosuke Niwa
Comment 5 2012-03-01 13:55:53 PST
Committed r109418: <http://trac.webkit.org/changeset/109418>
Note You need to log in before you can comment on or make changes to this bug.