Bug 114080 - Move the flakiness dashboard to the new instance at webkit-test-results.appspot.com
Summary: Move the flakiness dashboard to the new instance at webkit-test-results.appsp...
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: Ojan Vafai
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-05 21:13 PDT by Ojan Vafai
Modified: 2013-04-05 23:26 PDT (History)
3 users (show)

See Also:

Attachments
Patch (1.38 KB, patch)
2013-04-05 21:15 PDT, Ojan Vafai 		no flags 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 Ojan Vafai 2013-04-05 21:13:32 PDT 
Move the flakiness dashboard to the new instance at webkit-test-results.appspot.com
Comment 1 Ojan Vafai 2013-04-05 21:15:20 PDT 
Created attachment 196716 [details]
Patch
Comment 2 Ojan Vafai 2013-04-05 21:16:08 PDT 
BTW, I've already pushed this. So you can see it live at http://webkit-test-results.appspot.com.
Comment 3 Ojan Vafai 2013-04-05 21:17:31 PDT 
I added rniwa@webkit.org to the owners for the new appengine instance. Please let me know if there's some other address/account you'd like me to add instead.
Comment 4 Ryosuke Niwa 2013-04-05 21:40:35 PDT 
Can we modify the (In reply to comment #3)
> I added rniwa@webkit.org to the owners for the new appengine instance. Please let me know if there's some other address/account you'd like me to add instead.

Could you add rniwa@chromium.org instead? I don't have a google account on rniwa@webkit.org.  Is this account to be paid by Google?  Are you expecting either Apple or some other company will take it over at some point?
Comment 5 Ryosuke Niwa 2013-04-05 21:41:03 PDT 
Anyway, thanks for the follow up :)
Comment 6 Ryosuke Niwa 2013-04-05 21:43:34 PDT 
Also, I'm not sure if this was intentional but http://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html appears to be pulling results from test-results.appspot.com.
Comment 7 Ryosuke Niwa 2013-04-05 21:45:13 PDT 
(In reply to comment #4)
> (In reply to comment #3)
> > I added rniwa@webkit.org to the owners for the new appengine instance. Please let me know if there's some other address/account you'd like me to add instead.
> 
> Could you add rniwa@chromium.org instead? I don't have a google account on rniwa@webkit.org.

Never mind this. I've accepted the invitation with the right account.
Comment 8 Ojan Vafai 2013-04-05 21:53:09 PDT 
(In reply to comment #6)
> Also, I'm not sure if this was intentional but http://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html appears to be pulling results from test-results.appspot.com.

Yeah, I tried to cover this in the ChangeLog entry, until the build.webkit.org bots start uploading to webkit-test-results.appspot.com, we don't want to change the dashboard code to download from them.

(In reply to comment #4)
> Can we modify the (In reply to comment #3)
> > I added rniwa@webkit.org (:rniwa) (r) to the owners for the new appengine instance. Please let me know if there's some other address/account you'd like me to add instead.
> 
> Is this account to be paid by Google?

No.

>  Are you expecting either Apple or some other company will take it over at some point?

Yes.

Right now it's operating as a free app. If it starts hitting the free quota limit, it will start dropping requests. It's not clear to me if WebKit's use of the dashboard will exceed the free quota or not. Certainly, it won't exceed the quota right now when the json files are still being uploaded to and downloaded from test-results.appspot.com.
Comment 9 WebKit Commit Bot 2013-04-05 23:26:01 PDT 
Comment on attachment 196716 [details]
Patch

Clearing flags on attachment: 196716

Committed r147833: <http://trac.webkit.org/changeset/147833>
Comment 10 WebKit Commit Bot 2013-04-05 23:26:03 PDT 
All reviewed patches have been landed.  Closing bug.