commit 9d83d736beace7a8c307c1794e1869b09c82bbf2
author Derek Beckett <dbeckett@chromium.org> Mon May 02 15:21:16 2022 -0700
committer Derek Beckett <dbeckett@chromium.org> Tue May 03 16:00:16 2022 +0000
tree e9166f66fb06f111535a80bceb9e1222149afd58
parent 9f3d50fc5896e944135aded49ccadd8ad96b6168

cft: move cros-test to new builder, mark non-critical

New builder for cros-test, so it needs to go back to non-critical to not
block CQs/releases etc.

BUG=b:230875828
TEST=unittest, cq

Cq-Depend: chromium:3621940
Change-Id: I9e9161e3d9664bba0c31c9896d71348472243f88
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3620849
Reviewed-by: Jaques Clapauch <jaquesc@google.com>
Tested-by: Derek Beckett <dbeckett@chromium.org>
Commit-Queue: Derek Beckett <dbeckett@chromium.org>

api/controller/test.py

diff --git a/api/controller/test.py b/api/controller/test.py
index 986b724..e5349ea 100644
--- a/api/controller/test.py
+++ b/api/controller/test.py
@@ -161,8 +161,8 @@
         os.path.join(TEST_SERVICE_DIR, 'dut/docker/build-dockerimage.sh'),
     'cros-test':
         os.path.join(
-            PLATFORM_DEV_DIR,
-            'test/container/utils/cloudbuild-dockerimage.py'
+            TEST_SERVICE_DIR,
+            'python/src/docker_libs/cli/build-dockerimages.py.py'
         ),
     'cros-test-finder':
         os.path.join(
@@ -298,13 +298,20 @@
             )
         )
       else:
-        logging.debug('%s build failed.\nStdout:\n%s\nStderr:\n%s',
-                      human_name, cmd_result.stdout, cmd_result.stderr)
-        result.failure.CopyFrom(
-            test_pb2.TestServiceContainerBuildResult.Failure(
-                error_message=cmd_result.stdout
-            )
-        )
+        if human_name == 'cros-test':
+          logging.debug('%s build failed.\nStdout:\n%s\nStderr:\n%s',
+                        human_name, cmd_result.stdout, cmd_result.stderr)
+          result.success.CopyFrom(
+              test_pb2.TestServiceContainerBuildResult.Success()
+          )
+        else:
+          logging.debug('%s build failed.\nStdout:\n%s\nStderr:\n%s',
+                        human_name, cmd_result.stdout, cmd_result.stderr)
+          result.failure.CopyFrom(
+              test_pb2.TestServiceContainerBuildResult.Failure(
+                  error_message=cmd_result.stdout
+              )
+          )
       output_proto.results.append(result)