[telemetry] Ignore missing C-state info from powermetrics.

[telemetry] Ignore missing C-state info from powermetrics.

Our "new" Mac HDD bots have CPUs from, like, 2008, and C-states probably didn't exist back then.

BUG=517580
Committed: https://crrev.com/bdbbdf659ca40c0f62c39f0b1a22083ceb9f3311
Cr-Commit-Position: refs/heads/master@{#355326}

[dtu, 2015-10-21 16:12:56]

Description was changed from

==========
[telemetry] Ignore missing C-state info from powermetrics.

Our "new" Mac HDD bots have CPUs from, like, 2008, and C-states probably didn't
exist back then.


BUG=517580
==========

to

==========
[telemetry] Ignore missing C-state info from powermetrics.

Our "new" Mac HDD bots have CPUs from, like, 2008, and C-states probably didn't
exist back then.


BUG=517580
==========

[dtu, 2015-10-21 16:12:56]

dtu@chromium.org changed reviewers:
+ aiolos@chromium.org, eakuefner@chromium.org, nednguyen@google.com,
zhenw@chromium.org

[nednguyen, 2015-10-21 16:13:03]

lgtm

[aiolos (Not reviewing), 2015-10-21 16:13:17]

lgtm

[eakuefner, 2015-10-21 16:13:41]

lgtm

https://codereview.chromium.org/1419743003/diff/1/tools/telemetry/telemetry/i...
File
tools/telemetry/telemetry/internal/platform/power_monitor/powermetrics_power_monitor.py
(right):

https://codereview.chromium.org/1419743003/diff/1/tools/telemetry/telemetry/i...
tools/telemetry/telemetry/internal/platform/power_monitor/powermetrics_power_monitor.py:201:
# Old CPUs don't have c-states, so if data is missing, just ignore it.
logging.info?

[dtu, 2015-10-21 16:13:48]

The CQ bit was checked by dtu@chromium.org

[dtu, 2015-10-21 16:14:00]

The CQ bit was unchecked by dtu@chromium.org

[dtu, 2015-10-21 16:28:49]

The CQ bit was checked by dtu@chromium.org

[dtu, 2015-10-21 16:28:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
eakuefner@chromium.org, aiolos@chromium.org, nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/1419743003/#ps20001
(title: "Logging")

[dtu, 2015-10-21 16:29:13]

The CQ bit was unchecked by dtu@chromium.org

[dtu, 2015-10-21 16:30:05]

The CQ bit was checked by dtu@chromium.org

[dtu, 2015-10-21 16:30:06]

The patchset sent to the CQ was uploaded after l-g-t-m from
eakuefner@chromium.org, aiolos@chromium.org, nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/1419743003/#ps40001
(title: "Reword")

[commit-bot: I haz the power, 2015-10-21 16:30:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1419743003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1419743003/40001

[commit-bot: I haz the power, 2015-10-21 17:36:35]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-10-21 17:37:58]

Patchset 3 (id:??) landed as
https://crrev.com/bdbbdf659ca40c0f62c39f0b1a22083ceb9f3311
Cr-Commit-Position: refs/heads/master@{#355326}