Change in vdsm[master]: caps: coarse-grain libvirt.rpm may be missing

danken at redhat.com danken at redhat.com
Tue Oct 8 17:50:14 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

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. Instead, it should report the information of the installed
libvirt-daemon-kvm.rpm.

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 <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/19295
Reviewed-by: mooli tayer <mtayer at redhat.com>
Reviewed-by: Douglas Schilling Landgraf <dougsland at redhat.com>
---
M vdsm/caps.py
1 file changed, 10 insertions(+), 8 deletions(-)

Approvals:
  Douglas Schilling Landgraf: Looks good to me, but someone else must approve
  mooli tayer: Looks good to me, but someone else must approve
  Dan Kenigsberg: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I733a1a2411c910caa030e435c2a7f751a8e4dad4
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list