From 2d0aaf1689cef7e8722e9119f74ad4cb2a68145e Mon Sep 17 00:00:00 2001
From: Bruno Goncalves <bgoncalv@redhat.com>
Date: Feb 22 2018 10:42:53 +0000
Subject: standard-test-rhts: fail if restraint fails


And also report issues if phases return WARN or None result just to be sure.

This resolves https://stg.pagure.io/releng/fedora-scm-requests/issue/47

Signed-off-by: Miroslav Vadkerti <mvadkert@redhat.com>
Signed-off-by: Bruno Goncalves <bgoncalv@redhat.com>

---

diff --git a/roles/standard-test-rhts/tasks/main.yml b/roles/standard-test-rhts/tasks/main.yml
index 1284f69..1cd1760 100644
--- a/roles/standard-test-rhts/tasks/main.yml
+++ b/roles/standard-test-rhts/tasks/main.yml
@@ -226,6 +226,7 @@
     args:
       chdir: "{{ remote_artifacts }}/"
     ignore_errors: True
+    register: restraint_result
 
 - always:
   - name: Extract job output directory from restraint logfile
@@ -253,6 +254,6 @@
     when: artifacts|default("") != ""
 
   - name: Check the results for failures
-    shell: grep '^FAIL ' "{{ remote_artifacts }}/test.log"
+    shell: egrep '^(FAIL|WARN|None) ' "{{ remote_artifacts }}/test.log"
     register: test_fails
-    failed_when: test_fails.stdout or test_fails.stderr
+    failed_when: test_fails.stdout or test_fails.stderr or restraint_result.rc != 0