Change in vdsm[ovirt-3.6]: spec: Require newer qemu version for el7

nsoffer at redhat.com nsoffer at redhat.com
Thu Nov 26 11:10:25 UTC 2015


Nir Soffer has posted comments on this change.

Change subject: spec: Require newer qemu version for el7
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

-1 for visibility

https://gerrit.ovirt.org/#/c/49205/1/vdsm.spec.in
File vdsm.spec.in:

Line 229: Requires: qemu-kvm-rhev >= 10:2.1.2-23.el7_1.2
Line 230: Requires: qemu-img-rhev >= 10:2.1.2-23.el7_1.2
Line 231: %else
Line 232: Requires: qemu-kvm-rhev >= 10:2.3.0-13.el7
Line 233: Requires: qemu-img-rhev >= 10:2.3.0-13.el7
Fred, according to your comment:

 Freddy Rolland
 10:11 AM

 Patch Set 1: Verified+1
 [root at venus-vdsa ~]# rpm -qa | grep qemu
 qemu-img-rhev-2.3.0-31.el7.x86_64
 qemu-kvm-rhev-2.3.0-31.el7.x86_64
 qemu-kvm-tools-rhev-2.3.0-31.el7_2.1.x86_64

The version should be 2.3.0-31, but you require 10:2.3...

This was also in the old code - can you explain this difference?

Did you verify that vdsm can be installed on rhel 7.2 machine with these requirements?

This is the only way to verify the spec, checking the package exists is not good enough, you may have a syntax error in the spec.
Line 234: %endif # centos
Line 235: %else
Line 236: Requires: qemu-kvm >= 2:2.1.3-11
Line 237: Requires: qemu-img >= 2:2.1.3-11


-- 
To view, visit https://gerrit.ovirt.org/49205
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifd2ea13f318bce2c15cdd9acc65944eb9f2f604d
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list