Remove Win Aura bots from the buildbot and tryservers since they're now redundant as trunk Windows …

Remove Win Aura bots from the buildbot and tryservers since they're now redundant as trunk Windows is using Aura. 

BUG=316199
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=234668

[jam, 2013-11-08 01:05:48]

This is the removal part of win aura bots that was split off from
https://codereview.chromium.org/64723003. I'll hold off on landing this for a
while per the discussion on the original change.

[jam, 2013-11-12 16:07:30]

hey guys, looks like this is ready to land now?

[Paweł Hajdan Jr., 2013-11-12 18:54:59]

LGTM with comments.

https://codereview.chromium.org/66063002/diff/40001/masters/master.chromium.w...
File masters/master.chromium.win/master_win_cfg.py (right):

https://codereview.chromium.org/66063002/diff/40001/masters/master.chromium.w...
masters/master.chromium.win/master_win_cfg.py:403: 'events',
Is this addition intentional?

https://codereview.chromium.org/66063002/diff/40001/masters/master.tryserver....
File masters/master.tryserver.chromium/master.cfg (right):

https://codereview.chromium.org/66063002/diff/40001/masters/master.tryserver....
masters/master.tryserver.chromium/master.cfg:704: 'events_br',
Is this addition intentional?

[jam, 2013-11-12 19:00:08]

https://codereview.chromium.org/66063002/diff/40001/masters/master.chromium.w...
File masters/master.chromium.win/master_win_cfg.py (right):

https://codereview.chromium.org/66063002/diff/40001/masters/master.chromium.w...
masters/master.chromium.win/master_win_cfg.py:403: 'events',
On 2013/11/12 18:54:59, Paweł Hajdan Jr. wrote:
> Is this addition intentional?

yeah, this is after syncing because this target had since gotten added to
win_aura

https://codereview.chromium.org/66063002/diff/40001/masters/master.tryserver....
File masters/master.tryserver.chromium/master.cfg (right):

https://codereview.chromium.org/66063002/diff/40001/masters/master.tryserver....
masters/master.tryserver.chromium/master.cfg:704: 'events_br',
On 2013/11/12 18:54:59, Paweł Hajdan Jr. wrote:
> Is this addition intentional?

ditto

[commit-bot: I haz the power, 2013-11-12 19:01:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jam@chromium.org/66063002/40001

[commit-bot: I haz the power, 2013-11-12 19:01:57]

Presubmit check for 66063002-40001 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
tests/masters_test.py failed
1 masters failed:
master.tryserver.chromium
50 masters succeeded, 1 failed, 1 skipped in 11.6s.

=== Error running master.tryserver.chromium === 
2013-11-12 11:01:25-0800 [-] Creating BuildMaster -- buildbot.version: 0.8.4p1
2013-11-12 11:01:25-0800 [-] loading configuration from
/b/commit-queue/workdir/tools/build/masters/master.tryserver.chromium/master.cfg
2013-11-12 11:01:25-0800 [-] configuration update started
2013-11-12 11:01:26-0800 [-] unable to import dnotify, so Maildir will use
polling instead
2013-11-12 11:01:27-0800 [-] error while parsing config file
2013-11-12 11:01:27-0800 [-] Unhandled Error
	Traceback (most recent call last):
	  File
"/b/commit-queue/workdir/tools/build/third_party/buildbot_8_4p1/buildbot/master.py",
line 248, in loadTheConfigFile
	    d = self.loadConfig(f)
	  File
"/b/commit-queue/workdir/tools/build/third_party/buildbot_8_4p1/buildbot/master.py",
line 602, in loadConfig
	    d.addCallback(do_load)
	  File
"/b/commit-queue/workdir/tools/build/third_party/twisted_10_2/twisted/internet/defer.py",
line 297, in addCallback
	    callbackKeywords=kw)
	  File
"/b/commit-queue/workdir/tools/build/third_party/twisted_10_2/twisted/internet/defer.py",
line 286, in addCallbacks
	    self._runCallbacks()
	--- <exception caught here> ---
	  File
"/b/commit-queue/workdir/tools/build/third_party/twisted_10_2/twisted/internet/defer.py",
line 542, in _runCallbacks
	    current.result = callback(current.result, *args, **kw)
	  File
"/b/commit-queue/workdir/tools/build/third_party/buildbot_8_4p1/buildbot/master.py",
line 277, in do_load
	    exec f in localDict
	  File
"/b/commit-queue/workdir/tools/build/masters/master.tryserver.chromium/master.cfg",
line 2235, in <module>
	    master_utils.VerifySetup(c, slaves)
	  File "/b/commit-queue/workdir/tools/build/scripts/master/master_utils.py",
line 123, in VerifySetup
	    'referenced at all' % name)
	master.master_utils.InvalidConfig: Slave vm164-m4 defined in your slaves_list
is not referenced at all



tests/masters_cfg_test.py failed

FAILURE  The following master.cfg files did not load:

/b/commit-queue/workdir/tools/build/masters/master.tryserver.chromium
>  error while parsing
/b/commit-queue/workdir/tools/build/masters/master.tryserver.chromium/master.cfg:

>  Traceback (most recent call last):
>    File
"/b/commit-queue/workdir/tools/build/scripts/common/master_cfg_utils.py", line
104, in LoadConfig
>      localdict = ExecuteConfig(canonical_config)
>    File
"/b/commit-queue/workdir/tools/build/scripts/common/master_cfg_utils.py", line
79, in ExecuteConfig
>      exec f in localDict
>    File
"/b/commit-queue/workdir/tools/build/masters/master.tryserver.chromium/master.cfg",
line 2235, in <module>
>      master_utils.VerifySetup(c, slaves)
>    File "/b/commit-queue/workdir/tools/build/scripts/master/master_utils.py",
line 123, in VerifySetup
>      'referenced at all' % name)
>  InvalidConfig: Slave vm164-m4 defined in your slaves_list is not referenced
at all
>  
>  build error encountered! aborting build

Parsed 51 masters successfully, 1 failed, 1 skipped in 9.1s.


Presubmit checks took 40.4s to calculate.

[jam, 2013-11-12 19:09:56]

ptal to see the update to slaves.cfg which hopefully fixes the presubmit check

[ghost stip (do not use), 2013-11-12 19:19:24]

https://chromiumcodereview.appspot.com/66063002/diff/120001/masters/master.ch...
File masters/master.chromium.win/slaves.cfg (left):

https://chromiumcodereview.appspot.com/66063002/diff/120001/masters/master.ch...
masters/master.chromium.win/slaves.cfg:362: 'hostname': 'build60-m1',
put these slaves in master.chromium.unused

https://chromiumcodereview.appspot.com/66063002/diff/120001/masters/master.tr...
File masters/master.tryserver.chromium/slaves.cfg (left):

https://chromiumcodereview.appspot.com/66063002/diff/120001/masters/master.tr...
masters/master.tryserver.chromium/slaves.cfg:492: range(216, 219) + range(220,
223) + range(224, 227) +
can you move these to master.tryserver.unused?

[jam, 2013-11-12 19:32:27]

https://codereview.chromium.org/66063002/diff/120001/masters/master.chromium....
File masters/master.chromium.win/slaves.cfg (left):

https://codereview.chromium.org/66063002/diff/120001/masters/master.chromium....
masters/master.chromium.win/slaves.cfg:362: 'hostname': 'build60-m1',
On 2013/11/12 19:19:24, stip wrote:
> put these slaves in master.chromium.unused

Done, thanks didn't know about that

https://codereview.chromium.org/66063002/diff/120001/masters/master.tryserver...
File masters/master.tryserver.chromium/slaves.cfg (left):

https://codereview.chromium.org/66063002/diff/120001/masters/master.tryserver...
masters/master.tryserver.chromium/slaves.cfg:492: range(216, 219) + range(220,
223) + range(224, 227) +
On 2013/11/12 19:19:24, stip wrote:
> can you move these to master.tryserver.unused?

Done.

[ghost stip (do not use), 2013-11-12 20:06:48]

I think you meant master.chromium.unused instead of master.chromiumos.unused :)

lg other than that

https://chromiumcodereview.appspot.com/66063002/diff/250001/masters/master.ch...
File masters/master.chromiumos.unused/slaves.cfg (right):

https://chromiumcodereview.appspot.com/66063002/diff/250001/masters/master.ch...
masters/master.chromiumos.unused/slaves.cfg:27: 'master': 'ChromiumWin',
when you move this over to master.chromiumos.unused, change this to
ChromiumUnused

[jam, 2013-11-12 20:39:31]

On 2013/11/12 20:06:48, stip wrote:
> I think you meant master.chromium.unused instead of master.chromiumos.unused
:)

good catch, doh. fixed

https://codereview.chromium.org/66063002/diff/250001/masters/master.chromiumo...
File masters/master.chromiumos.unused/slaves.cfg (right):

https://codereview.chromium.org/66063002/diff/250001/masters/master.chromiumo...
masters/master.chromiumos.unused/slaves.cfg:27: 'master': 'ChromiumWin',
On 2013/11/12 20:06:48, stip wrote:
> when you move this over to master.chromiumos.unused, change this to
> ChromiumUnused

Done.

[ghost stip (do not use), 2013-11-13 00:11:00]

lgtm

[commit-bot: I haz the power, 2013-11-13 00:11:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jam@chromium.org/66063002/290001

[commit-bot: I haz the power, 2013-11-13 00:12:39]

Change committed as 234668