ci/lava: Color red for fatal and yellow for warning
authorGuilherme Gallo <guilherme.gallo@collabora.com>
Thu, 7 Jul 2022 02:22:09 +0000 (23:22 -0300)
committerMarge Bot <emma+marge@anholt.net>
Fri, 8 Jul 2022 12:26:05 +0000 (12:26 +0000)
Fatal errors now have red foreground color and retry messages yellow
one.

Signed-off-by: Guilherme Gallo <guilherme.gallo@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17389>

.gitlab-ci/lava/lava_job_submitter.py
.gitlab-ci/lava/utils/log_follower.py

index 82a52a7..1bbd77d 100755 (executable)
@@ -357,17 +357,13 @@ def fetch_logs(job, max_idle_time, log_follower) -> None:
     # silence means that the device has died and we should try it again
     if datetime.now() - job.last_log_time > max_idle_time:
         max_idle_time_min = max_idle_time.total_seconds() / 60
-        print_log(
-            f"{CONSOLE_LOG['BOLD']}"
-            f"{CONSOLE_LOG['FG_RED']}"
-            f"No log output for {max_idle_time_min} minutes; "
-            "assuming device has died, retrying"
-            f"{CONSOLE_LOG['RESET']}"
-        )
 
         raise MesaCITimeoutError(
+            f"{CONSOLE_LOG['BOLD']}"
+            f"{CONSOLE_LOG['FG_YELLOW']}"
             f"LAVA job {job.job_id} does not respond for {max_idle_time_min} "
-            "minutes. Retry.",
+            "minutes. Retry."
+            f"{CONSOLE_LOG['RESET']}",
             timeout_duration=max_idle_time,
         )
 
index d08923b..c61a9aa 100644 (file)
@@ -207,7 +207,10 @@ def print_log(msg):
 
 
 def fatal_err(msg):
-    print_log(msg)
+    colored_msg = f"{CONSOLE_LOG['FG_RED']}"
+    f"{msg}"
+    f"{CONSOLE_LOG['RESET']}"
+    print_log(colored_msg)
     sys.exit(1)