modules/fm-orchestrator
1
0
Fork 0
mirror of https://pagure.io/fm-orchestrator.git synced 2026-02-11 17:14:59 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Log the original exception in consumer before trying to do anything else. Also commit the db.session before doing build.transition

Browse Source
This commit is contained in:
Jan Kaluza
2017-10-20 14:47:49 +02:00
parent 60b3d97c97
commit 8fedf35b6c
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
module_build_service/scheduler/consumer.py
View File

@@ -238,12 +238,14 @@ class MBSConsumer(fedmsg.consumers.FedmsgConsumer):
try:
further_work = handler(conf, session, msg) or []
except Exception as e:
if build:
build.transition(conf, state=models.BUILD_STATES['failed'],
state_reason=str(e))
msg = 'Could not process message handler. See the traceback.'
log.exception(msg)
session.commit()
session.rollback()
if build:
session.refresh(build)
build.transition(conf, state=models.BUILD_STATES['failed'],
state_reason=str(e))
session.commit()
log.debug("Done with %s" % idx)