Hello Douglas Schilling Landgraf,
I'd like you to do a code review. Please visit
to review the following change.
Change subject: Report qemu-kvm-rhev if installed ......................................................................
Report qemu-kvm-rhev if installed
Change-Id: If6b59241bbf38b6cd15fd941e1f68115853d22c1 Bug-Url: https://bugzilla.redhat.com/1029792 Signed-off-by: Dan Kenigsberg danken@redhat.com Reviewed-on: http://gerrit.ovirt.org/21226 Reviewed-by: Douglas Schilling Landgraf dougsland@redhat.com --- M vdsm/caps.py 1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/64/21364/1
diff --git a/vdsm/caps.py b/vdsm/caps.py index c5ecadc..a6e22d2 100644 --- a/vdsm/caps.py +++ b/vdsm/caps.py @@ -324,8 +324,8 @@ pkgs = {'kernel': kernelDict()}
if getos() in (OSName.RHEVH, OSName.OVIRT, OSName.FEDORA, OSName.RHEL): - KEY_PACKAGES = {'qemu-kvm': ('qemu-kvm',), - 'qemu-img': ('qemu-img',), + KEY_PACKAGES = {'qemu-kvm': ('qemu-kvm', 'qemu-kvm-rhev'), + 'qemu-img': ('qemu-img', 'qemu-img-rhev'), 'vdsm': ('vdsm',), 'spice-server': ('spice-server',), 'libvirt': ('libvirt', 'libvirt-daemon-kvm'),
Douglas Schilling Landgraf has posted comments on this change.
Change subject: Report qemu-kvm-rhev if installed ......................................................................
Patch Set 1: Code-Review+1
Humble Devassy Chirammal has posted comments on this change.
Change subject: Report qemu-kvm-rhev if installed ......................................................................
Patch Set 1: Code-Review+1
Yaniv Bronhaim has posted comments on this change.
Change subject: Report qemu-kvm-rhev if installed ......................................................................
Patch Set 2: Verified+1 Code-Review+2
Yaniv Bronhaim has submitted this change and it was merged.
Change subject: Report qemu-kvm-rhev if installed ......................................................................
Report qemu-kvm-rhev if installed
Change-Id: If6b59241bbf38b6cd15fd941e1f68115853d22c1 Bug-Url: https://bugzilla.redhat.com/1029792 Signed-off-by: Dan Kenigsberg danken@redhat.com Reviewed-on: http://gerrit.ovirt.org/21226 Reviewed-by: Douglas Schilling Landgraf dougsland@redhat.com Reviewed-on: http://gerrit.ovirt.org/21364 Tested-by: Yaniv Bronhaim ybronhei@redhat.com Reviewed-by: Yaniv Bronhaim ybronhei@redhat.com --- M vdsm/caps.py 1 file changed, 2 insertions(+), 2 deletions(-)
Approvals: Yaniv Bronhaim: Verified; Looks good to me, approved
vdsm-patches@lists.fedorahosted.org