Dan Kenigsberg has submitted this change and it was merged.

Change subject: Add soft dependency on libvirt's metadata support to stats 
collector
......................................................................


Add soft dependency on libvirt's metadata support to stats collector

There is an issue with libvirt on CentOS 6. The available version does
not support part of the metadata API. This generates an error message
every 15 seconds.

This patch workarounds this by logging the error only once.

Revert once libvirt supports the xml element metadata on CentOS.

See bug: https://bugzilla.redhat.com/show_bug.cgi?id=1115039

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1114968
Change-Id: I3d38d3aff3bda42dfaafa87bab9fa5983fb4b9a8
Signed-off-by: Martin Sivak <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/29504
Reviewed-by: Francesco Romani <[email protected]>
Reviewed-by: Nir Soffer <[email protected]>
Reviewed-by: Michal Skrivanek <[email protected]>
Reviewed-by: Dan Kenigsberg <[email protected]>
(cherry picked from commit 935123db785d0a848cfeae5e9cf8cc75ad598701)
Reviewed-on: http://gerrit.ovirt.org/29788
---
M vdsm/virt/vm.py
1 file changed, 25 insertions(+), 4 deletions(-)

Approvals:
  Martin Sivák: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3d38d3aff3bda42dfaafa87bab9fa5983fb4b9a8
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Martin Sivák <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Martin Sivák <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: [email protected]
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to