RESOLVED FIXED 111166
Update list of builders for test results server
https://bugs.webkit.org/show_bug.cgi?id=111166
Summary Update list of builders for test results server
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
Patch (103.85 KB, patch)
2013-03-05 00:48 PST, jochen
no flags
jochen
Comment 1 2013-03-01 05:15:06 PST
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
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.