Bug 111166

Summary: Update list of builders for test results server
Product: WebKit Reporter: jochen
Component: New BugsAssignee: jochen
Status: RESOLVED FIXED    
Severity: Normal CC: ojan, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

jochen
Reported 2013-03-01 05:14:53 PST
Add WebKit (Content Shell) Android builder to test-results server config
Attachments
Patch (1.33 KB, patch)
2013-03-01 05:15 PST, jochen 		no flags
DetailsFormatted DiffDiff
Patch (103.85 KB, patch)
2013-03-05 00:48 PST, jochen 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
jochen
Comment 1 2013-03-01 05:15:06 PST
Created attachment 190939 [details] Patch
jochen
Comment 2 2013-03-01 05:15:36 PST
Ojan, can you have a look please?
Ojan Vafai
Comment 3 2013-03-01 11:16:31 PST
Comment on attachment 190939 [details] Patch builders.jsonp is auto generated. Please use the script to generate it. Eventually, we won't even be checking in this file. We'll just be running the script directly on the appengine server.
jochen
Comment 4 2013-03-04 23:38:29 PST
ok, i'll just wait until the server auto generates it
Ojan Vafai
Comment 5 2013-03-05 00:28:56 PST
(In reply to comment #4) > ok, i'll just wait until the server auto generates it I think I miscommunicated. This file is generated by Tools/TestResultServer/generate_builders_json.py. Right now you have to run it manually and commit the result. Eventually, I'd like to run it on the server and not check it into the repo at all. Noone is actively working on making it auto-generated on the server though. So, in the meantime, you can run it. I can also r+ this patch as is if you want. In theory, the next time someone runs the python script it should do the right thing regardless. :) Up to you.
jochen
Comment 6 2013-03-05 00:48:03 PST
Created attachment 191425 [details] Patch
jochen
Comment 7 2013-03-05 00:48:48 PST
ok, seems like this wasn't the only bot that needed updating
Ojan Vafai
Comment 8 2013-03-05 10:46:47 PST
Comment on attachment 191425 [details] Patch Thanks!
WebKit Review Bot
Comment 9 2013-03-05 11:07:30 PST
Comment on attachment 191425 [details] Patch Clearing flags on attachment: 191425 Committed r144783: <http://trac.webkit.org/changeset/144783>
WebKit Review Bot
Comment 10 2013-03-05 11:07:33 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.