openstack/ceilometer
Code Issues Proposed changes

Merge "Don't check libvirt exception if libvirt is absent"

Browse Source
This commit is contained in:
Jenkins
2017-07-11 20:33:23 +00:00
committed by Gerrit Code Review
parent df50841b52 c1ffbb4019
commit 7471bbf3a7
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
ceilometer/compute/virt/libvirt/utils.py
View File

@@ -92,11 +92,13 @@ def refresh_libvirt_connection(conf, klass):
connection = getattr(klass, '_libvirt_connection', None) connection = getattr(klass, '_libvirt_connection', None)
if not connection or not connection.isAlive(): if not connection or not connection.isAlive():
connection = new_libvirt_connection(conf) connection = new_libvirt_connection(conf)
setattr(klass, '_libvirt_connection', connection) setattr(klass, '_libvirt_connection', connection)
return connection return connection
def is_disconnection_exception(e): def is_disconnection_exception(e):
if not libvirt:
return False
return (isinstance(e, libvirt.libvirtError) return (isinstance(e, libvirt.libvirtError)
and e.get_error_code() in (libvirt.VIR_ERR_SYSTEM_ERROR, and e.get_error_code() in (libvirt.VIR_ERR_SYSTEM_ERROR,
libvirt.VIR_ERR_INTERNAL_ERROR) libvirt.VIR_ERR_INTERNAL_ERROR)