Change in vdsm[master]: spec: require qemu-*-ev on CentOS >= 7

amureini at redhat.com amureini at redhat.com
Fri May 8 12:10:27 UTC 2015


Allon Mureinik has posted comments on this change.

Change subject: spec: require qemu-*-ev on CentOS >= 7
......................................................................


Patch Set 1:

(2 comments)

Nir, I don't understand this comment at all.

EL builds of qemu lack all live features (snapshots, merge, storage migration, etc). This is a known fact, and up to now we used to provied qemu-kvm-rhev in oVirt's repo (see http://resources.ovirt.org/pub/ovirt-3.5/rpm/el7Server/x86_64/).
The -ev builds, which are provided upstream by the CentOS virtualization SIG are the standard way of consuming enterprise features on EL outside of Red Hat's product matrix, and is already being consumed by other similar projects (e.g., OpenStack).

https://gerrit.ovirt.org/#/c/40670/1//COMMIT_MSG
Commit Message:

Line 5: CommitDate: 2015-05-07 17:54:00 +0300
Line 6: 
Line 7: spec: require qemu-*-ev on CentOS >= 7
Line 8: 
Line 9: Added a requirement for qemu-kvm-ev and qemu-img-ev on CentOS >= 7.
> please explain why it is good (because -ev supply features that we want!) a
CBS link for when you update the commit msg:
http://cbs.centos.org/koji/packageinfo?packageID=539
Line 10: 
Line 11: Change-Id: Ibbec719863d7b12150574a60d071f38bcfa62398


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

Line 276: Requires: systemd >= 197-1.fc18.2
Line 277: %endif
Line 278: 
Line 279: %global rhev_suffix -rhev
Line 280: %global ev_suffix -ev
> AFAIK, *-ev packages, much like *-rhev ones are not available in koji. Has 
This is for upstream EL builds.
the -ev packages are part of the Centos virt SIG (see, e.g., http://cbs.centos.org/koji/packageinfo?packageID=539) and we deliver them in oVirt's repo too (see, e.g., under http://resources.ovirt.org/pub/ovirt-master-snapshot/rpm/el7Server/x86_64/)
Line 281: 
Line 282: %if 0%{?rhel}
Line 283: %if 0%{?rhel} >= 7
Line 284: %if 0%{?rhev_build}


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibbec719863d7b12150574a60d071f38bcfa62398
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list