Bug 110954 - Stop uploading results to webkit-perf.appspot.com
Summary: Stop uploading results to webkit-perf.appspot.com
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
URL:
Keywords:
Depends on:
Blocks: 77037 111148
  Show dependency tree / graph
 
Reported: 2013-02-27 00:31 PST by Ryosuke Niwa
Modified: 2013-03-16 01:20 PDT (History)
6 users (show)

See Also:

Attachments
Stop uploading results to webkit-perf.appspot.com (8.82 KB, patch)
2013-02-27 00:34 PST, Ryosuke Niwa 		benjamin: review+
Details | Formatted Diff | Diff
Updates buildbot (1.46 KB, patch)
2013-02-27 00:44 PST, Ryosuke Niwa 		benjamin: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2013-02-27 00:31:24 PST 
Stop uploading results to webkit-perf.appspot.com
Comment 1 Ryosuke Niwa 2013-02-27 00:34:37 PST 
Created attachment 190458 [details]
Stop uploading results to webkit-perf.appspot.com
Comment 2 Ryosuke Niwa 2013-02-27 00:35:31 PST 
Note that we have to land this patch first before we can update build.webkit.org to use perf.webkit.org as the test results server.
Comment 3 Ryosuke Niwa 2013-02-27 00:44:50 PST 
Created attachment 190460 [details]
Updates buildbot
Comment 4 Ryosuke Niwa 2013-02-27 00:45:53 PST 
(In reply to comment #3)
> Created an attachment (id=190460) [details]
> Updates buildbot

Again, this patch can only be landed after the first patch is landed and all bots have caught up to that revision.
Comment 5 Benjamin Poulain 2013-02-27 00:47:33 PST 
Comment on attachment 190458 [details]
Stop uploading results to webkit-perf.appspot.com

View in context: https://bugs.webkit.org/attachment.cgi?id=190458&action=review

> Tools/Scripts/webkitpy/performance_tests/perftestsrunner.py:226
> +            if options.test_results_server == 'webkit-perf.appspot.com':

When does that happen? Some bots hardcode this?
Comment 6 Benjamin Poulain 2013-02-27 00:48:04 PST 
Comment on attachment 190460 [details]
Updates buildbot

Yay!
Comment 7 Ryosuke Niwa 2013-02-27 00:48:58 PST 
Comment on attachment 190458 [details]
Stop uploading results to webkit-perf.appspot.com

View in context: https://bugs.webkit.org/attachment.cgi?id=190458&action=review

>> Tools/Scripts/webkitpy/performance_tests/perftestsrunner.py:226
>> +            if options.test_results_server == 'webkit-perf.appspot.com':
> 
> When does that happen? Some bots hardcode this?

It's hard-coded in build.webkit.org.
Comment 8 Ryosuke Niwa 2013-02-27 00:58:16 PST 
Committed r144155: <http://trac.webkit.org/changeset/144155>
Comment 9 Ryosuke Niwa 2013-02-27 00:59:56 PST 
I still need to land the second patch.
Comment 10 Ryosuke Niwa 2013-03-16 01:20:39 PDT 
Landed as http://trac.webkit.org/changeset/144252.