Change in mom[master]: logging: fix the exception handler

alitke at redhat.com alitke at redhat.com
Mon Feb 16 14:47:29 UTC 2015


Adam Litke has submitted this change and it was merged.

Change subject: logging: fix the exception handler
......................................................................


logging: fix the exception handler

In the exception handler of the MOM class constructor, there
are two issues

- one, major, is about the name of the logging reference, which
  wasn't updated. This will lead to MOM blow in the error (recovery!)
  path.
- one, minor, is about the old way to log exceptions, using
  the exc_info=True argument.

This patch fixes both.

Change-Id: I58741b7e430180c969f5596575e8b8461de49076
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/37816
Reviewed-by: Adam Litke <alitke at redhat.com>
---
M mom/__init__.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Adam Litke: Looks good to me, approved
  Francesco Romani: Verified



-- 
To view, visit http://gerrit.ovirt.org/37816
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I58741b7e430180c969f5596575e8b8461de49076
Gerrit-PatchSet: 2
Gerrit-Project: mom
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list