Bug 35944 – Add an appengine app to host chromium layout test results

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Victor Wang]

Add an appengine app to host layout test results (JSON) for chromium flakiness dashboard. The app could be used to host other test result files if needed.

[Victor Wang]

Created attachment 50348 [details]
Proposed Patch

[WebKit Review Bot]

Attachment 50348 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebKitTools/Scripts/webkitpy/test_result_server/index.yaml:1:  One or more unexpected \r (^M) found; better to use only a \n  [whitespace/carriage_return] [1]
Suppressing further [whitespace/carriage_return] reports for this file.
Total errors found: 30 in 12 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Victor Wang]

Created attachment 50351 [details]
Fixed a CR in proposed patch

[Adam Barth]

Comment on attachment 50351 [details]
Fixed a CR in proposed patch

This should go in a subdirectory of WebKitTools, analogous to QueueStatusServer, which is another of our AppEngine apps.

I'm not sure we want svn:executable or svn:eol-style set on these files.

+ class DeleteFile

This app looks wide open for anonymous folks on the interest to screw with out datastore.  Do we care?

Do we want some CSS to make this pretty?  We can do that in another patch.

[Victor Wang]

Created attachment 50632 [details]
Proposed patch

A couple of updates in new patch:
-. moved the appengine to WebKitTool
-. added CSS
-. removed svn:executable and svn:eol-style
-. added test type support
-. more blobstore upload redirect handling.

Eventually, we will add code to limit users who could update these files, for now, I would like to get a version in and start testing it.

[Adam Barth]

Comment on attachment 50632 [details]
Proposed patch

Okiedokes.  Theoretically we're supposed to keep our python to 80 cols, but we probably won't be able to enforce that until we get our automatic pep8 stye checker.

[WebKit Commit Bot]

Comment on attachment 50632 [details]
Proposed patch

Clearing flags on attachment: 50632

Committed r55960: <http://trac.webkit.org/changeset/55960>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.