From ed34226544fc10d6b4571314111af2f52a66fae5 Mon Sep 17 00:00:00 2001
From: Erik Reid <erik.reid@geant.org>
Date: Thu, 17 Feb 2022 11:18:52 +0100
Subject: [PATCH] re-raise unhandled exceptions after logging

---
 inventory_provider/tasks/worker.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/inventory_provider/tasks/worker.py b/inventory_provider/tasks/worker.py
index 098fb57f..03d1b7c5 100644
--- a/inventory_provider/tasks/worker.py
+++ b/inventory_provider/tasks/worker.py
@@ -567,6 +567,7 @@ def reload_lab_router_config_chorded(self, hostname):
         logger.exception(errmsg)
         update_latch_status(InventoryTask.config, pending=True, failure=True)
         self.log_error(errmsg)
+        raise e
 
 
 @app.task(base=InventoryTask, bind=True, name='reload_router_config')
@@ -600,11 +601,12 @@ def reload_router_config_chorded(self, hostname):
             snmp_refresh_peerings_chorded(hostname, community, logical_systems)
 
         logger.info(f'updated configuration for {hostname}')
-    except:
+    except Exception as e:
         errmsg = f'unhandled exception loading {hostname} info'
         logger.exception(errmsg)
         update_latch_status(InventoryTask.config, pending=True, failure=True)
         self.log_error(errmsg)
+        raise e
 
 
 def retrieve_and_persist_netconf_config(
-- 
GitLab