From 3ab1371ce33288b46f1add81a45287aafa680613 Mon Sep 17 00:00:00 2001
From: Piotr Konopka <piotr.jan.konopka@cern.ch>
Date: Thu, 30 Nov 2023 16:51:30 +0100
Subject: [PATCH] Move trg.errorcleanup role to before_GO_ERROR (#626)

Since in a healthy end of run sequence we perform end of trigger at before_STOP_ACTIVITY, there is no reason to perform it later in the sequence in case of going to ERROR.
I confirmed with the trigger and reconstruction experts that the change is OK.
---
 workflows/readout-dataflow.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/workflows/readout-dataflow.yaml b/workflows/readout-dataflow.yaml
index 8d0062d19..ec4865636 100755
--- a/workflows/readout-dataflow.yaml
+++ b/workflows/readout-dataflow.yaml
@@ -1972,7 +1972,7 @@ roles:
       - name: errorcleanup
         call:
           func: trg.Cleanup()
-          trigger: after_GO_ERROR-100
+          trigger: before_GO_ERROR
           timeout: "{{ trg_cleanup_timeout }}"
           critical: false
       - name: destroycleanup