Merge pull request #854 from nathanchance/handle-internal-errors-better

Handle internal errors better

Author: nathanchance

.github/problem-matchers/internal.json

@@ -0,0 +1,16 @@
+{
+    "problemMatcher": [
+        {
+            "__comment_owner": "internal continuous integration errors",
+            "owner": "internal",
+            "pattern": [
+                {
+                    "__comment_regexp1": "[CI ERROR] Build is not finished on TuxSuite's side!",
+                    "regexp": "^\\[CI (WARNING|ERROR)\\]\\s+(.*)$",
+                    "severity": 1,
+                    "message": 2
+                }
+            ]
+        }
+    ]
+}

scripts/check-logs.py

@@ -58,7 +58,12 @@ def verify_build():
     retries = 0
     max_retries = 9
     while retries < max_retries:
-        if build["tuxbuild_status"] == "complete":
+        # build never started
+        if (build_status := build['tuxbuild_status']) == 'error':
+            print_red(f"msg from tuxsuite: {build['status_message']}")
+            sys.exit(1)
+        # build is finished
+        elif build_status == "complete":
             break
 
         if retries:

utils.py

@@ -290,12 +290,12 @@ def update_repository_variable(
 
 
 def print_red(msg):
-    print(f"\033[91m{msg}\033[0m", file=sys.stderr)
+    print(f"\033[91m[CI ERROR] {msg}\033[0m", file=sys.stderr)
     sys.stderr.flush()
 
 
 def print_yellow(msg):
-    print(f"\033[93m{msg}\033[0m", file=sys.stdout)
+    print(f"\033[93m[CI WARNING] {msg}\033[0m", file=sys.stdout)
     sys.stdout.flush()