From cadb0587c14b1abd473e46306977c74a2f36309c Mon Sep 17 00:00:00 2001
From: Valerij Maljulin <vmaljuli@redhat.com>
Date: Dec 10 2018 15:24:11 +0000
Subject: Non-zero return code for failing builds when building locally

Fixes #1063

Signed-off-by: Valerij Maljulin <vmaljuli@redhat.com>

---

diff --git a/module_build_service/manage.py b/module_build_service/manage.py
index 0f753eb..091e573 100755
--- a/module_build_service/manage.py
+++ b/module_build_service/manage.py
@@ -150,7 +150,7 @@ def build_module_locally(local_build_nsvs=None, yaml_file=None, stream=None, ski
             handle = FileStorage(fd)
             handle.filename = filename
             try:
-                submit_module_build_from_yaml(
+                modules_list = submit_module_build_from_yaml(
                     username, handle, str(stream), skiptests, optional_params)
             except StreamAmbigous as e:
                 logging.error(str(e))
@@ -163,6 +163,9 @@ def build_module_locally(local_build_nsvs=None, yaml_file=None, stream=None, ski
         # Run the consumer until stop_condition returns True
         module_build_service.scheduler.main([], stop)
 
+        if any(module.state == models.BUILD_STATES['failed'] for module in modules_list):
+            raise RuntimeError('Module build failed')
+
 
 @console_script_help
 @manager.command