Remove Skia's forked buildbot code

Remove Skia's forked buildbot code

I'm keeping slave_hosts_cfg information for now, as well as everything used by
the slaves at launch.

BUG=skia:761, skia:2431
R=rmistry@google.com

Committed: https://skia.googlesource.com/buildbot/+/90441aadfde6070f685adbbbfd917947cd17b261

[borenet, 2014-10-13 20:56:09]

borenet@google.com changed reviewers:
+ rmistry@google.com

[borenet, 2014-10-13 20:56:16]

Will submit after the new buildbot world seems stable.

https://codereview.chromium.org/648353002/diff/380002/DEPS
File DEPS (left):

https://codereview.chromium.org/648353002/diff/380002/DEPS#oldcode56
DEPS:56: 
I think that no other code uses these DEPS...

[borenet, 2014-10-17 15:01:05]

PTAL.  I think this is ready to go now that the RecreateSKPs bot has been
ported.

[rmistry, 2014-10-17 16:53:00]

LGTM

[borenet, 2014-10-17 18:04:18]

PTAL at patch set 5.

https://codereview.chromium.org/648353002/diff/2090001/scripts/launch_slaves.py
File scripts/launch_slaves.py (right):

https://codereview.chromium.org/648353002/diff/2090001/scripts/launch_slaves....
scripts/launch_slaves.py:169: print 'Successfully launched slave %s.' %
self._slavename
We lose keepalive_conditions as part of this change.  They're still listed in
the slaves.cfg file on the masters, but in order to make that work we'd have to
read the files for each master from googlesource.  That's doable (we already do
it in the slave_hosts_cfg test), but I didn't bother quite yet.

https://codereview.chromium.org/648353002/diff/2090001/site_config/slave_host...
File site_config/slave_hosts_cfg.py (right):

https://codereview.chromium.org/648353002/diff/2090001/site_config/slave_host...
site_config/slave_hosts_cfg.py:93: ('skiabot-shuttle-ubuntu12-gtx550ti-001',
'0', False),
The bool in this column has changed meaning from "connects to new master" to
"uses internal code"

[rmistry, 2014-10-17 18:19:21]

LGTM with one nit

https://codereview.chromium.org/648353002/diff/2090001/scripts/launch_slaves.py
File scripts/launch_slaves.py (right):

https://codereview.chromium.org/648353002/diff/2090001/scripts/launch_slaves....
scripts/launch_slaves.py:169: print 'Successfully launched slave %s.' %
self._slavename
On 2014/10/17 18:04:18, borenet wrote:
> We lose keepalive_conditions as part of this change.  They're still listed in
> the slaves.cfg file on the masters, but in order to make that work we'd have
to
> read the files for each master from googlesource.  That's doable (we already
do
> it in the slave_hosts_cfg test), but I didn't bother quite yet.

Acknowledged.

https://codereview.chromium.org/648353002/diff/2090001/scripts/launch_slaves....
scripts/launch_slaves.py:184: copies: dictionary with 'source' and 'destination'
keys whose values are the
Can you replace copies with slave_num.

[borenet, 2014-10-17 18:39:26]

https://codereview.chromium.org/648353002/diff/2090001/scripts/launch_slaves.py
File scripts/launch_slaves.py (right):

https://codereview.chromium.org/648353002/diff/2090001/scripts/launch_slaves....
scripts/launch_slaves.py:184: copies: dictionary with 'source' and 'destination'
keys whose values are the
On 2014/10/17 18:19:21, rmistry wrote:
> Can you replace copies with slave_num.

Done.

[borenet, 2014-10-17 18:42:18]

Committed patchset #6 (id:2100001) manually as
90441aadfde6070f685adbbbfd917947cd17b261 (presubmit successful).