From 9adcd7649d8a20f30a0a04e98e6d89c79400c299 Mon Sep 17 00:00:00 2001
From: Simone Spinelli <simone.spinelli@geant.org>
Date: Thu, 20 Jul 2023 14:21:52 +0000
Subject: [PATCH] Modify the failure form to make it more user-friendly

---
 gso/services/provisioning_proxy.py | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/gso/services/provisioning_proxy.py b/gso/services/provisioning_proxy.py
index 3ccf0bd7..a9405e02 100644
--- a/gso/services/provisioning_proxy.py
+++ b/gso/services/provisioning_proxy.py
@@ -213,19 +213,16 @@ def _confirm_pp_results(state: State) -> FormGenerator:
                 f"completed, please confirm the results below."
             )
 
-        run_status: str = ReadOnlyField(state["pp_run_results"]["status"])
         if not successful_run:
-            warning_run_status: str = ReadOnlyField(state["pp_run_results"]["status"])
-            failed_run_status: str = ReadOnlyField(state["pp_run_results"]["status"])
-            broken_run_status: str = ReadOnlyField(state["pp_run_results"]["status"])
-            ouch_run_status: str = ReadOnlyField(state["pp_run_results"]["status"])
-            unsuccessful_run_status: str = ReadOnlyField(state["pp_run_results"]["status"])
+            pp_retry_label1: Label = (
+                    "Provisioning Proxy - playbook execution failed: inspect the output before proceeding"  # type: ignore
+                    )
+        run_status: str = ReadOnlyField(state["pp_run_results"]["status"])
         run_results: LongText = ReadOnlyField(f"{state['pp_run_results']['output']}")
         if not successful_run:
             pp_retry_label: Label = (
-                "Provisioning Proxy playbook execution failed, it will be retried (up to two times)."  # type: ignore
+                "Click submit to retry. Otherwise, abort the workflow from the process tab."  # type: ignore
             )
-        confirm: Accept = Accept("INCOMPLETE")
 
     yield ConfirmRunPage
 
-- 
GitLab