From 11d9fa7148d95f0669273bac72adeb3860f4c091 Mon Sep 17 00:00:00 2001 From: Jan Kaluza Date: Fri, 7 Apr 2017 07:22:22 +0200 Subject: [PATCH] Do not raise exception in poller, but log it instead. --- module_build_service/scheduler/producer.py | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/module_build_service/scheduler/producer.py b/module_build_service/scheduler/producer.py index b09a2dab..b97d6379 100644 --- a/module_build_service/scheduler/producer.py +++ b/module_build_service/scheduler/producer.py @@ -42,13 +42,17 @@ class MBSProducer(PollingProducer): def poll(self): with models.make_session(conf) as session: - self.log_summary(session) - # XXX: detect whether it's actually stuck first - # self.process_waiting_module_builds(session) - self.process_open_component_builds(session) - self.fail_lost_builds(session) - self.process_paused_module_builds(conf, session) - self.trigger_new_repo_when_stalled(conf, session) + try: + self.log_summary(session) + # XXX: detect whether it's actually stuck first + # self.process_waiting_module_builds(session) + self.process_open_component_builds(session) + self.fail_lost_builds(session) + self.process_paused_module_builds(conf, session) + self.trigger_new_repo_when_stalled(conf, session) + except Exception as e: + msg = 'Error in poller execution:' + log.exception(msg) log.info('Poller will now sleep for "{}" seconds' .format(conf.polling_interval))