commit 6d718d6e8f55428259654f8ede9dbaf3d5b54fb5
author Alex Klein <saklein@chromium.org> Wed Jan 18 15:55:51 2023 -0700
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Jan 20 20:55:49 2023 +0000
tree 3964c1605c5c70c1f8147969abd412d2464f8dcd
parent 247d792fecd46b9b2085494f575f290cf9c617f5

api: controller/sysroot: PackageIndexInfo deserialization

Switch PackageIndexInfo deserialization from the class methods to
the controller_util functions. These are the only usages of the
protobuf [de]serialization interface outside of tests.

BUG=b:265998356
TEST=./run_tests

Change-Id: I272807e611490289dd11103ee142a9984510aec5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4179354
Tested-by: Alex Klein <saklein@chromium.org>
Reviewed-by: Lizzy Presland <zland@google.com>
Commit-Queue: Alex Klein <saklein@chromium.org>

api/controller/sysroot.py

diff --git a/api/controller/sysroot.py b/api/controller/sysroot.py
index b8a3369..e65123a 100644
--- a/api/controller/sysroot.py
+++ b/api/controller/sysroot.py
@@ -14,7 +14,6 @@
 from chromite.api.controller import controller_util
 from chromite.api.gen.chromiumos import common_pb2
 from chromite.api.metrics import deserialize_metrics_log
-from chromite.lib import binpkg
 from chromite.lib import build_target_lib
 from chromite.lib import chroot_lib
 from chromite.lib import cros_build_lib
@@ -175,7 +174,7 @@
         input_proto.build_target, input_proto.profile
     )
     package_indexes = [
-        binpkg.PackageIndexInfo.from_protobuf(x)
+        controller_util.deserialize_package_index_info(x)
         for x in input_proto.package_indexes
     ]
     run_configs = sysroot.SetupBoardRunConfig(
@@ -306,7 +305,7 @@
     ]
 
     package_indexes = [
-        binpkg.PackageIndexInfo.from_protobuf(x)
+        controller_util.deserialize_package_index_info(x)
         for x in input_proto.package_indexes
     ]