Remove recipe for ozone_ecs

Remove recipe for ozone_ecs

ozone_ecs buildbots were removed.
https://codereview.chromium.org/1920133002

* remove an arg used only in ozone_ecs recipe from chromium.runtest

BUG=

Committed: https://chromium.googlesource.com/chromium/tools/build/+/6739408c1342b8ed2f229325a80aafe6403b9fd1

[tikuta, 2016-11-01 04:51:56]

The CQ bit was checked by tikuta@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-01 04:52:03]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-01 05:01:42]

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

[commit-bot: I haz the power, 2016-11-01 05:01:43]

Dry run: This issue passed the CQ dry run.

[tikuta, 2016-11-01 05:04:23]

The CQ bit was checked by tikuta@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-01 05:04:32]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tikuta, 2016-11-01 05:09:36]

Description was changed from

==========
use_goma_module=True for ozone_ecs buildbots

BUG=
==========

to

==========
Remove recipe for ozone_ecs

ozone_ecs buildbots were removed.
https://codereview.chromium.org/1920133002

* remove an arg used only in ozone_ecs recipe from chromium.runtest

BUG=
==========

[tikuta, 2016-11-01 05:09:36]

tikuta@chromium.org changed reviewers:
+ dpranke@chromium.org, phajdan.jr@chromium.org

[commit-bot: I haz the power, 2016-11-01 05:13:42]

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

[commit-bot: I haz the power, 2016-11-01 05:13:43]

Dry run: This issue passed the CQ dry run.

[Dirk Pranke, 2016-11-01 16:45:29]

lgtm

[Paweł Hajdan Jr., 2016-11-02 10:06:40]

https://codereview.chromium.org/2466823002/diff/20001/scripts/slave/recipe_mo...
File scripts/slave/recipe_modules/chromium/api.py (left):

https://codereview.chromium.org/2466823002/diff/20001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium/api.py:429:
full_args.append('--no-spawn-dbus')
Can we also remove this flag from runtest.py?

Consider verifying that nothing internal depends on that. Quick check on
cs.chromium.org shows ecs was the only thing in public repos.

[tikuta, 2016-11-04 02:43:40]

The CQ bit was checked by tikuta@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-04 02:43:50]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tikuta, 2016-11-04 02:51:00]

https://codereview.chromium.org/2466823002/diff/20001/scripts/slave/recipe_mo...
File scripts/slave/recipe_modules/chromium/api.py (left):

https://codereview.chromium.org/2466823002/diff/20001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium/api.py:429:
full_args.append('--no-spawn-dbus')
On 2016/11/02 10:06:40, Paweł Hajdan Jr. wrote:
> Can we also remove this flag from runtest.py?
> 
> Consider verifying that nothing internal depends on that. Quick check on
> http://cs.chromium.org shows ecs was the only thing in public repos.

Done.

I ran `fgrep 'no-spawn-dbus' . -rsnI` in build_internal and confirmed that only
ozone_ecs recipes use this option.

Do I need to take care below?
https://cs.chromium.org/chromium/src/infra/scripts/legacy/scripts/slave/runte...

[commit-bot: I haz the power, 2016-11-04 02:54:00]

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

[commit-bot: I haz the power, 2016-11-04 02:54:01]

Dry run: This issue passed the CQ dry run.

[Paweł Hajdan Jr., 2016-11-04 07:16:20]

LGTM

Yes, please also remove the flag from chromium/src copy.

[tikuta, 2016-11-04 07:41:54]

The CQ bit was checked by tikuta@chromium.org

[tikuta, 2016-11-04 07:41:54]

The patchset sent to the CQ was uploaded after l-g-t-m from dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/2466823002/#ps40001
(title: "remove --no-spawn-dbus from runtest.py")

[commit-bot: I haz the power, 2016-11-04 07:42:06]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-04 07:52:03]

Description was changed from

==========
Remove recipe for ozone_ecs

ozone_ecs buildbots were removed.
https://codereview.chromium.org/1920133002

* remove an arg used only in ozone_ecs recipe from chromium.runtest

BUG=
==========

to

==========
Remove recipe for ozone_ecs

ozone_ecs buildbots were removed.
https://codereview.chromium.org/1920133002

* remove an arg used only in ozone_ecs recipe from chromium.runtest

BUG=

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/6739408c1342b8ed2f22...
==========

[commit-bot: I haz the power, 2016-11-04 07:52:04]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/tools/build/+/6739408c1342b8ed2f22...