From 0218eb3aa7c7c692b58807154d3b7367463bd2cd Mon Sep 17 00:00:00 2001
From: swenzel <sandro.wenzel@cern.ch>
Date: Thu, 26 Jun 2025 10:56:37 +0200
Subject: [PATCH] anchorMC: Don't execute TPC timeseries if there was a failure

---
 MC/run/ANCHOR/anchorMC.sh | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/MC/run/ANCHOR/anchorMC.sh b/MC/run/ANCHOR/anchorMC.sh
index 64f7669c1..21e07fa57 100755
--- a/MC/run/ANCHOR/anchorMC.sh
+++ b/MC/run/ANCHOR/anchorMC.sh
@@ -368,15 +368,16 @@ echo_info "Ready to start main workflow"
 
 ${O2DPG_ROOT}/MC/bin/o2_dpg_workflow_runner.py -f workflow.json -tt ${ALIEN_JDL_O2DPGWORKFLOWTARGET:-aod} --cpu-limit ${ALIEN_JDL_CPULIMIT:-8} --dynamic-resources
 MCRC=$?  # <--- we'll report back this code
-if [[ "${ALIEN_JDL_ADDTIMESERIESINMC}" != "0" ]]; then
+if [[ "${MCRC}" == "0" && "${ALIEN_JDL_ADDTIMESERIESINMC}" != "0" ]]; then
   # Default value is 1 so this is run by default.
   echo_info "Running TPC time series"
   ${O2DPG_ROOT}/MC/bin/o2_dpg_workflow_runner.py -f workflow.json -tt tpctimes
+  # Note: We could maybe avoid this if-else by including `tpctimes` directly in the workflow-targets above
 fi
 
-[[ ! -z "${DISABLE_QC}" ]] && echo_info "QC is disabled, skip it."
+[[ -n "${DISABLE_QC}" ]] && echo_info "QC is disabled, skip it."
 
-if [[ -z "${DISABLE_QC}" && "${MCRC}" = "0" && "${remainingargs}" == *"--include-local-qc"* ]] ; then
+if [[ -z "${DISABLE_QC}" && "${MCRC}" == "0" && "${remainingargs}" == *"--include-local-qc"* ]] ; then
   # do QC tasks
   echo_info "Doing QC"
   ${O2DPG_ROOT}/MC/bin/o2_dpg_workflow_runner.py -f workflow.json --target-labels QC --cpu-limit ${ALIEN_JDL_CPULIMIT:-8} -k