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@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 <froll...@redhat.com> Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com> Gerrit-Reviewer: Fred Rolland <froll...@redhat.com> Gerrit-Reviewer: Freddy Rolland <froll...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches