openstack/ceilometer
Code Issues Proposed changes

Revert "Add hypervisor inspector sanity check"

Browse Source 
This change have introduced an issue, when the libvirt unix socket
is created after Ceilometer start. The pollster is disabled when
it should wait that libvirt come up.

The intent of the original bug was only to have a cosmetic logs
when the configuration file is wrong.

Also I don't see a safe way to disable the pollster when we can't
connect to libvirt, the reason is always unknown from Ceilometer PoV.

This reverts commit d17e75d267.

Change-Id: I63479d19fcf4a99fb198859c2e6ae370ff0e24e1
This commit is contained in:
Mehdi Abaakouk
2016-11-28 13:14:54 +01:00
committed by Mehdi Abaakouk (sileht)
parent f8d1f602a7
commit 72b5a1e2bc
5 changed files with 0 additions and 49 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -83,10 +83,6 @@ class LibvirtInspector(virt_inspector.Inspector):
return self._connection
def check_sanity(self):
if not self.connection:
raise virt_inspector.NoSanityException()
@retry_on_disconnect
def _lookup_by_uuid(self, instance):
instance_name = util.instance_name(instance)