Disable the CPU tracing agent on Windows

Disable the CPU tracing agent on Windows

This will allow us to reenable it on other platforms.

BUG=chromium:647443
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/42836761f6ea7edc8dc9bb3ded09c305f3baee45

[charliea (OOO until 10-5), 2016-10-26 21:15:35]

charliea@chromium.org changed reviewers:
+ nednguyen@google.com

[rnephew (Reviews Here), 2016-10-26 21:16:53]

rnephew@chromium.org changed reviewers:
+ rnephew@chromium.org

[rnephew (Reviews Here), 2016-10-26 21:16:53]

https://codereview.chromium.org/2453683004/diff/1/telemetry/telemetry/interna...
File telemetry/telemetry/internal/platform/tracing_agent/cpu_tracing_agent.py
(right):

https://codereview.chromium.org/2453683004/diff/1/telemetry/telemetry/interna...
telemetry/telemetry/internal/platform/tracing_agent/cpu_tracing_agent.py:162:
return (os_name in ['mac', 'linux'])
Should there be a todo with the psutil bug listed in it?

[charliea (OOO until 10-5), 2016-10-26 21:27:53]

https://codereview.chromium.org/2453683004/diff/1/telemetry/telemetry/interna...
File telemetry/telemetry/internal/platform/tracing_agent/cpu_tracing_agent.py
(right):

https://codereview.chromium.org/2453683004/diff/1/telemetry/telemetry/interna...
telemetry/telemetry/internal/platform/tracing_agent/cpu_tracing_agent.py:162:
return (os_name in ['mac', 'linux'])
On 2016/10/26 21:16:53, rnephew (Reviews Here) wrote:
> Should there be a todo with the psutil bug listed in it?

Good call. Done.

[nednguyen, 2016-10-26 21:34:44]

lgtm

[nednguyen, 2016-10-26 21:35:30]

On 2016/10/26 21:34:44, nednguyen wrote:
> lgtm

Note that its output on Linux & Mac is currently missing data because of
crbug.com/640215

[charliea (OOO until 10-5), 2016-11-01 15:15:32]

The CQ bit was checked by charliea@chromium.org

[commit-bot: I haz the power, 2016-11-01 15:15:44]

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 15:40:34]

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

[commit-bot: I haz the power, 2016-11-01 15:40:35]

Try jobs failed on following builders:
  Catapult Windows Tryserver on master.tryserver.client.catapult (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Wi...)

[charliea (OOO until 10-5), 2016-11-01 17:35:32]

The CQ bit was checked by charliea@chromium.org

[charliea (OOO until 10-5), 2016-11-01 17:35:33]

The patchset sent to the CQ was uploaded after l-g-t-m from nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/2453683004/#ps40001
(title: "Disable tests on Windows also")

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

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 17:56:19]

Description was changed from

==========
Disable the CPU tracing agent on Windows

This will allow us to reenable it on other platforms.

BUG=chromium:647443
==========

to

==========
Disable the CPU tracing agent on Windows

This will allow us to reenable it on other platforms.

BUG=chromium:647443

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-11-01 17:56:20]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...