commit c7d647fc0af62c13cab44185a6f0a4b23076cf31
author Alex Klein <saklein@chromium.org> Mon Jan 06 12:00:48 2020 -0700
committer Commit Bot <commit-bot@chromium.org> Tue Jan 07 20:55:26 2020 +0000
tree a1e035dcc35d9f280fd85e83758241ab762c74c3
parent 57ad64bcf3302063e7d6dcf3f61002fd90a31eb9

Build API: Stop parsing chroot goma message.

BUG=chromium:1038386
TEST=run_tests

Change-Id: I57374d3d52dbc63a2d1949104896f2477d429ddf
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/1988626
Tested-by: Alex Klein <saklein@chromium.org>
Reviewed-by: Michael Mortensen <mmortensen@google.com>
Reviewed-by: David Burger <dburger@chromium.org>
Commit-Queue: Alex Klein <saklein@chromium.org>

api/controller/controller_util.py

diff --git a/api/controller/controller_util.py b/api/controller/controller_util.py
index 5306f80..ac15052 100644
--- a/api/controller/controller_util.py
+++ b/api/controller/controller_util.py
@@ -25,12 +25,11 @@
   """Invalid message."""
 
 
-def ParseChroot(chroot_message, parse_goma=True):
+def ParseChroot(chroot_message):
   """Create a chroot object from the chroot message.
 
   Args:
     chroot_message (common_pb2.Chroot): The chroot message.
-    parse_goma (bool): Whether to try to parse the goma configs.
 
   Returns:
     Chroot: The parsed chroot object.
@@ -58,11 +57,8 @@
   if features:
     env['FEATURES'] = ' '.join(features)
 
-  goma = ParseGomaConfig(chroot_message.goma, path) if parse_goma else None
-
   chroot = Chroot(
       path=path, cache_dir=cache_dir, chrome_root=chrome_root, env=env)
-  chroot.goma = goma
 
   return chroot