Avoid key error for custom machine types

Avoid key error for custom machine types

BUG=

Review-Url: https://codereview.chromium.org/2801033003
Committed: https://github.com/luci/luci-py/commit/23f369607252c4d23b7cf2ac74df96c2ca2f4372

[kjlubick, 2017-04-07 12:13:47]

kjlubick@google.com changed reviewers:
+ maruel@chromium.org

[kjlubick, 2017-04-07 12:13:48]

We noticed quarantine errors like https://paste.googleplex.com/4977214210703360

[M-A Ruel, 2017-04-07 12:18:26]

The CQ bit was checked by maruel@chromium.org

[M-A Ruel, 2017-04-07 12:18:26]

lgtm

Thanks, I had told Ravi that I'd fix it in an hour but you beat me to it.

[commit-bot: I haz the power, 2017-04-07 12:18:31]

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

[borenet, 2017-04-07 12:19:25]

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

[borenet, 2017-04-07 12:19:26]

https://codereview.chromium.org/2801033003/diff/1/appengine/swarming/swarming...
File appengine/swarming/swarming_bot/api/os_utilities.py (right):

https://codereview.chromium.org/2801033003/diff/1/appengine/swarming/swarming...
appengine/swarming/swarming_bot/api/os_utilities.py:415: machine_cost =
GCE_MACHINE_COST_HOUR_US.get(machine_type, 0.)
I suspect we'll want a default, eg. GCE_MACHINE_COST_HOUR_US['n1-standard-8']

[M-A Ruel, 2017-04-07 12:20:16]

https://codereview.chromium.org/2801033003/diff/1/appengine/swarming/swarming...
File appengine/swarming/swarming_bot/api/os_utilities.py (right):

https://codereview.chromium.org/2801033003/diff/1/appengine/swarming/swarming...
appengine/swarming/swarming_bot/api/os_utilities.py:415: machine_cost =
GCE_MACHINE_COST_HOUR_US.get(machine_type, 0.)
On 2017/04/07 12:19:25, borenet wrote:
> I suspect we'll want a default, eg. GCE_MACHINE_COST_HOUR_US['n1-standard-8']

Bah, I prefer 0 than something really wrong. I don't mind much.

[commit-bot: I haz the power, 2017-04-07 12:22:43]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491567506042720, "parent_rev":
"abf57eb833a8dfa3731f23692cbe41be762de03c", "commit_rev":
"23f369607252c4d23b7cf2ac74df96c2ca2f4372"}

[commit-bot: I haz the power, 2017-04-07 12:22:48]

Description was changed from

==========
Avoid key error for custom machine types

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

to

==========
Avoid key error for custom machine types

BUG=

Review-Url: https://codereview.chromium.org/2801033003
Committed:
https://github.com/luci/luci-py/commit/23f369607252c4d23b7cf2ac74df96c2ca2f4372
==========

[commit-bot: I haz the power, 2017-04-07 12:22:49]

Committed patchset #1 (id:1) as
https://github.com/luci/luci-py/commit/23f369607252c4d23b7cf2ac74df96c2ca2f4372