From 41f51e9b60b82bc943a24fe05ec822555a8a39b2 Mon Sep 17 00:00:00 2001
From: Jorrit Schaap <schaap@astron.nl>
Date: Wed, 14 Sep 2016 13:53:40 +0000
Subject: [PATCH] Task #9607: set task to error if estimator fails

---
 .../ResourceAssigner/lib/assignment.py                   | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/SAS/ResourceAssignment/ResourceAssigner/lib/assignment.py b/SAS/ResourceAssignment/ResourceAssigner/lib/assignment.py
index a85e30fcb72..d399f5f049b 100755
--- a/SAS/ResourceAssignment/ResourceAssigner/lib/assignment.py
+++ b/SAS/ResourceAssignment/ResourceAssigner/lib/assignment.py
@@ -207,8 +207,13 @@ class ResourceAssigner():
             logger.info('skipping resource assignment for CEP4 task otdb_id=%s because status=%s' % (otdb_id, status))
             return
 
-        needed = self.getNeededResouces(specification_tree)
-        logger.info('doAssignment: getNeededResouces=%s' % (needed,))
+        try:
+            needed = self.getNeededResouces(specification_tree)
+            logger.info('doAssignment: getNeededResouces=%s' % (needed,))
+        except Exception as e:
+            logger.error(e)
+            self.radbrpc.updateTask(taskId, status='error')
+            self._sendNotification(task, 'error')
 
         if not str(otdb_id) in needed:
             logger.error("no otdb_id %s found in estimator results %s" % (otdb_id, needed))
-- 
GitLab