From 1bdd2f4b9a7f24864f273c8b6498acdba7f5d8ed Mon Sep 17 00:00:00 2001
From: Marcel Loose <loose@astron.nl>
Date: Thu, 6 Dec 2012 10:21:52 +0000
Subject: [PATCH] Task #3877: Minor fix to let vdsmaker bail out early, if none
of the makevds processes succeeded. Avoids incomprehensible error message
from combinevds.
---
CEP/Pipeline/recipes/sip/master/vdsmaker.py | 3 +++
1 file changed, 3 insertions(+)
diff --git a/CEP/Pipeline/recipes/sip/master/vdsmaker.py b/CEP/Pipeline/recipes/sip/master/vdsmaker.py
index 077cd0d1436..95af2e33541 100644
--- a/CEP/Pipeline/recipes/sip/master/vdsmaker.py
+++ b/CEP/Pipeline/recipes/sip/master/vdsmaker.py
@@ -107,6 +107,9 @@ class vdsmaker(BaseRecipe, RemoteCommandRecipeMixIn):
vds for vds, job in zip(vdsnames, jobs)
if job.results['returncode'] == 0
]
+ if not vdsnames:
+ self.logger.error("All makevds processes failed. Bailing out!")
+ return 1
# *********************************************************************
# 3. Combine VDS files to produce GDS
--
GitLab