add wm_unittests to tryserver win8_aura

Add wm_unittests to win8_aura tryserver

This is a short-term fix to bring bots in sync with waterfall.

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

[iannucci, 2014-06-09 20:09:49]

Actually I think you'll want to use _br here (buildrunner). stip@ has more info.

[Sergey Berezin, 2014-06-09 23:51:28]

I'll defer to other reviewers on this one.

[ghost stip (do not use), 2014-06-10 01:51:38]

lgtm. note that this doesn't require a tryserver restart

[chromium-reviews, 2014-06-10 01:53:48]

I thought any changes in master.cfg require restart. Would you elaborate?

Sheng

> On Jun 9, 2014, at 6:51 PM, "stip@chromium.org" <stip@chromium.org> wrote:
>
> lgtm. note that this doesn't require a tryserver restart
>
> https://chromiumcodereview.appspot.com/323943003/

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[ghost stip (do not use), 2014-06-10 01:55:07]

the point of buildrunner is that these changes don't need a restart. consider
buildrunner to be the precursor to recipes

[sheyang, 2014-06-10 15:45:31]

The CQ bit was checked by sheyang@chromium.org

[commit-bot: I haz the power, 2014-06-10 15:46:03]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sheyang@chromium.org/323943003/20001

[commit-bot: I haz the power, 2014-06-10 15:47:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-10 15:47:59]

Presubmit check for 323943003-20001 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
scripts/slave/unittests/recipe_simulation_test.py (8.89s) failed
.........................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................M..................

======================================================================
MISSING: chromium_android:example.perf_runner_basic
(/b/infra_internal/commit_queue/workdir/tools/build/scripts/slave/recipe_modules/chromium_android/example.expected/perf_runner_basic.json)
----------------------------------------------------------------------


----------------------------------------------------------------------
Ran 524 tests in 7.044s

FAILED (missing=1)


Presubmit checks took 110.3s to calculate.

Was the presubmit check useful? If not, run "git cl presubmit -v"
to figure out which PRESUBMIT.py was run, then run git blame
on the file to figure out who to ask for help.

[sheyang, 2014-06-10 16:04:04]

The CQ bit was checked by sheyang@chromium.org

[commit-bot: I haz the power, 2014-06-10 16:05:02]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sheyang@chromium.org/323943003/20001

[commit-bot: I haz the power, 2014-06-10 16:05:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-10 16:05:52]

Presubmit check for 323943003-20001 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
scripts/slave/unittests/recipe_simulation_test.py (8.80s) failed
....................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................M.......................

======================================================================
MISSING: chromium_android:example.perf_runner_basic
(/b/infra_internal/commit_queue/workdir/tools/build/scripts/slave/recipe_modules/chromium_android/example.expected/perf_runner_basic.json)
----------------------------------------------------------------------


----------------------------------------------------------------------
Ran 524 tests in 5.982s

FAILED (missing=1)


Presubmit checks took 45.6s to calculate.

[sheyang, 2014-06-10 17:44:35]

The CQ bit was checked by sheyang@chromium.org

[commit-bot: I haz the power, 2014-06-10 17:45:06]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sheyang@chromium.org/323943003/20001

[commit-bot: I haz the power, 2014-06-10 17:46:00]

Change committed as 276062

[Sergey Berezin, 2014-06-10 22:20:24]

On 2014/06/10 01:55:07, stip wrote:
> the point of buildrunner is that these changes don't need a restart. consider
> buildrunner to be the precursor to recipes

I can understand this for chromium_factory.py, but I believe we still need a
restart for master.cfg change, because we are adding a new test to the config.
Unless we add the test to testfilter in CQ, and then master.cfg change is
irrelevant?

[sheyang, 2014-06-10 22:28:51]

On 2014/06/10 22:20:24, Sergey Berezin wrote:
> On 2014/06/10 01:55:07, stip wrote:
> > the point of buildrunner is that these changes don't need a restart.
consider
> > buildrunner to be the precursor to recipes
> 
> I can understand this for chromium_factory.py, but I believe we still need a
> restart for master.cfg change, because we are adding a new test to the config.
> Unless we add the test to testfilter in CQ, and then master.cfg change is
> irrelevant?

I think runbuild.py will periodically read master.cfg to update the buildrunner
tests
dynamically.

I can see the wm_unittests added successfully: 
http://build.chromium.org/p/tryserver.chromium/builders/win8_aura/builds/2636