Small improvements to api.platforms.android.

Small improvements to api.platforms.android.

- Save the adb RSA keys, the public part will be pushed later.
- Stop silently ignoring broken RSA keys.
- Stop accepting dual call to android.initialize().
- Remove CpuScalingGovernor and use the actual string instead. It will be used
  to verify against the ones published by the device's kernel in a follow up.

R=stip@chromium.org
BUG=chromium:525873
Committed: https://github.com/luci/luci-py/commit/6274d54e173c4bf8875a97db47212ba927a6d9e8

[Marc-Antoine Ruel (Google), 2015-10-08 02:04:56]

Patchset #1 (id:1) has been deleted

[Marc-Antoine Ruel (Google), 2015-10-08 02:05:00]

Patchset #1 (id:20001) has been deleted

[Marc-Antoine Ruel (Google), 2015-10-08 02:06:10]

maruel@google.com changed reviewers:
+ maruel@google.com

[Marc-Antoine Ruel (Google), 2015-10-08 02:06:11]

Starting to see the light at the end of the tunnel, the bots are now stable
again, minor change CL to reduce the size of the next one.

[M-A Ruel, 2015-10-08 14:47:55]

Patchset #3 (id:70001) has been deleted

[M-A Ruel, 2015-10-08 14:48:08]

Patchset #3 (id:90001) has been deleted

[M-A Ruel, 2015-10-08 14:48:32]

maruel@chromium.org changed reviewers:
- maruel@google.com

[ghost stip (do not use), 2015-10-08 20:13:29]

these should have been sharded into separate CLs. review latency is your
bottleneck, and that increases exponentially with CL size :)

https://chromiumcodereview.appspot.com/1396603003/diff/110001/appengine/swarm...
File appengine/swarming/swarming_bot/api/platforms/android.py (right):

https://chromiumcodereview.appspot.com/1396603003/diff/110001/appengine/swarm...
appengine/swarming/swarming_bot/api/platforms/android.py:344: def
get_devices(bot=None):  # pylint: disable=unused-argument
?

https://chromiumcodereview.appspot.com/1396603003/diff/110001/appengine/swarm...
appengine/swarming/swarming_bot/api/platforms/android.py:405: def
all_values(cls):
I don't know why you're doing all this. why not just use the string values
directly?

https://chromiumcodereview.appspot.com/1396603003/diff/110001/appengine/swarm...
appengine/swarming/swarming_bot/api/platforms/android.py:538: parts =
line.split(u': ', 2)
I would have preferred this to be a separate CL to make it easier to review

[M-A Ruel, 2015-10-13 17:09:07]

Patchset #4 (id:130001) has been deleted

[M-A Ruel, 2015-10-13 17:20:52]

Patchset #4 (id:150001) has been deleted

[M-A Ruel, 2015-10-13 17:30:17]

Patchset #4 (id:170001) has been deleted

[M-A Ruel, 2015-10-13 17:30:34]

Trimmed the CL down.

[ghost stip (do not use), 2015-10-13 21:06:31]

lgtm

[ghost stip (do not use), 2015-10-14 06:56:24]

The CQ bit was checked by stip@chromium.org

[commit-bot: I haz the power, 2015-10-14 06:56:35]

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

[commit-bot: I haz the power, 2015-10-14 06:58:20]

Committed patchset #4 (id:190001) as
https://github.com/luci/luci-py/commit/6274d54e173c4bf8875a97db47212ba927a6d9e8