Dan Kenigsberg has uploaded a new change for review.

Change subject: caps: coarse-grain libvirt.rpm may be missing
......................................................................

caps: coarse-grain libvirt.rpm may be missing

As of http://gerrit.ovirt.org/15761 we no longer require the legacy
libvirt.rpm to be installed on Fedora, so getCaps should not look for
it.

Without this patch we get an annoying
    (_getKeyPackages) rpm package libvirt not found
log message, and do not report libvirt's version.

Change-Id: I733a1a2411c910caa030e435c2a7f751a8e4dad4
Signed-off-by: Dan Kenigsberg <dan...@redhat.com>
---
M vdsm/caps.py
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/95/19295/1

diff --git a/vdsm/caps.py b/vdsm/caps.py
index 3a7a6a2..ceea429 100644
--- a/vdsm/caps.py
+++ b/vdsm/caps.py
@@ -331,6 +331,9 @@
                         'mom': 'mom',
                         }
 
+        if getos() in (OSName.OVIRT, OSName.FEDORA):
+            KEY_PACKAGES['libvirt-daemon-kvm']
+
         if _glusterEnabled:
             KEY_PACKAGES.update(GLUSTER_RPM_PACKAGES)
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I733a1a2411c910caa030e435c2a7f751a8e4dad4
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to