config_service: Fixed bug regarding None entries in latest_revisions dict.

config_service: Fixed bug regarding None entries in latest_revisions dict.

Using the include_files option on config sets that had no latest revision
number caused a 503 Internal Server Error because there was no check for
None entries in the result from get_latest_revisions_async. This cl fixes
that by adding a check.

BUG=740736
Review-Url: https://codereview.chromium.org/2981473002
Committed: https://github.com/luci/luci-py/commit/32ed948dea48cdeeb11ee7b4d8ab0465837f42c9

[cwpayton, 2017-07-11 00:29:23]

Description was changed from

==========
config_service: Fixed bug regarding None entries in latest_revisions dict.

Using the include_files option on config sets that had no latest revision
number caused a 503 Internal Server Error because there was no check for
None entries in the result from get_latest_revisions_async. This cl fixes
that by adding a check.

BUG=730832
==========

to

==========
config_service: Fixed bug regarding None entries in latest_revisions dict.

Using the include_files option on config sets that had no latest revision
number caused a 503 Internal Server Error because there was no check for
None entries in the result from get_latest_revisions_async. This cl fixes
that by adding a check.

BUG=730832
==========

[cwpayton, 2017-07-11 00:29:24]

cwpayton@google.com changed reviewers:
+ nodir@chromium.org

[cwpayton, 2017-07-11 00:29:33]

PTAL

[nodir, 2017-07-11 21:40:49]

lgtm

[cwpayton, 2017-07-11 21:41:41]

The CQ bit was checked by cwpayton@google.com

[commit-bot: I haz the power, 2017-07-11 21:41:47]

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, 2017-07-11 21:44:20]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1499809301644940, "parent_rev":
"2f3e515849d142570e559710ff7c31370dc2fb20", "commit_rev":
"32ed948dea48cdeeb11ee7b4d8ab0465837f42c9"}

[commit-bot: I haz the power, 2017-07-11 21:44:22]

Description was changed from

==========
config_service: Fixed bug regarding None entries in latest_revisions dict.

Using the include_files option on config sets that had no latest revision
number caused a 503 Internal Server Error because there was no check for
None entries in the result from get_latest_revisions_async. This cl fixes
that by adding a check.

BUG=730832
==========

to

==========
config_service: Fixed bug regarding None entries in latest_revisions dict.

Using the include_files option on config sets that had no latest revision
number caused a 503 Internal Server Error because there was no check for
None entries in the result from get_latest_revisions_async. This cl fixes
that by adding a check.

BUG=730832

Review-Url: https://codereview.chromium.org/2981473002
Committed:
https://github.com/luci/luci-py/commit/32ed948dea48cdeeb11ee7b4d8ab0465837f42c9
==========

[commit-bot: I haz the power, 2017-07-11 21:44:24]

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

[nodir, 2017-07-12 00:43:03]

the bug should have been 740736

[cwpayton, 2017-07-12 00:43:52]

Description was changed from

==========
config_service: Fixed bug regarding None entries in latest_revisions dict.

Using the include_files option on config sets that had no latest revision
number caused a 503 Internal Server Error because there was no check for
None entries in the result from get_latest_revisions_async. This cl fixes
that by adding a check.

BUG=730832

Review-Url: https://codereview.chromium.org/2981473002
Committed:
https://github.com/luci/luci-py/commit/32ed948dea48cdeeb11ee7b4d8ab0465837f42c9
==========

to

==========
config_service: Fixed bug regarding None entries in latest_revisions dict.

Using the include_files option on config sets that had no latest revision
number caused a 503 Internal Server Error because there was no check for
None entries in the result from get_latest_revisions_async. This cl fixes
that by adding a check.

BUG=740736

Review-Url: https://codereview.chromium.org/2981473002
Committed:
https://github.com/luci/luci-py/commit/32ed948dea48cdeeb11ee7b4d8ab0465837f42c9
==========