service: simplify determine_android_version
This func was written to accept multiple boards and assert that the
version was the same for all of them, but then only ever called with
a single board at a time. Even the unittests patched it as if the
API only took one board. Drop the multiple board support to make it
look like the other android funcs in here and to simplify.
BUG=chromium:1051161
TEST=CQ passes
Change-Id: I516c1b2cc1792b676836f86b047eb5e96c9b95f8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/2739374
Reviewed-by: Michael Mortensen <mmortensen@google.com>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
diff --git a/api/controller/packages.py b/api/controller/packages.py
index 93ce232..dc9d29a 100644
--- a/api/controller/packages.py
+++ b/api/controller/packages.py
@@ -160,7 +160,7 @@
"""Returns the target versions."""
build_target = controller_util.ParseBuildTarget(input_proto.build_target)
# Android version.
- android_version = packages.determine_android_version([build_target.name])
+ android_version = packages.determine_android_version(build_target.name)
logging.info('Found android version: %s', android_version)
if android_version:
output_proto.android_version = android_version
@@ -221,7 +221,7 @@
build_target_metadata = output_proto.build_target_metadata.add()
build_target_metadata.build_target = build_target.name
# Android version.
- android_version = packages.determine_android_version([build_target.name])
+ android_version = packages.determine_android_version(build_target.name)
logging.info('Found android version: %s', android_version)
if android_version:
build_target_metadata.android_container_version = android_version