commit 6e97bc08904d07faac95005298c729071ff883e8
author Benjamin Shai <bshai@google.com> Mon Mar 07 21:34:39 2022 +0000
committer Commit Bot <commit-bot@chromium.org> Tue Mar 08 18:59:05 2022 +0000
tree 240ddbf589243cd789df7bb1278980281c0ed1d0
parent 42faf86187bde9c513aaf005550c90095a32338f

build-api: Update miniOS return code

In order to be able to detect miniOS payloads being skipped, we need the
failure_reason, which is only parsed if we use the return code that
indicates a response is available.

BUG=b:217557435
TEST=unit-test

Change-Id: I680ea962b6ced7b66e9cd21816e2c2441d171126
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/3508259
Reviewed-by: Greg Edelston <gredelston@google.com>
Reviewed-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Benjamin Shai <bshai@google.com>
Tested-by: Benjamin Shai <bshai@google.com>

api/controller/payload.py

diff --git a/api/controller/payload.py b/api/controller/payload.py
index c1b9042..015245d 100644
--- a/api/controller/payload.py
+++ b/api/controller/payload.py
@@ -102,6 +102,8 @@
   _SetGeneratePayloadOutputProto(output_proto, local_path, remote_uri)
   if remote_uri or input_proto.dryrun and local_path:
     return controller.RETURN_CODE_SUCCESS
+  elif output_proto.failure_reason:
+    return controller.RETURN_CODE_UNSUCCESSFUL_RESPONSE_AVAILABLE
   else:
     return controller.RETURN_CODE_COMPLETED_UNSUCCESSFULLY