Hello Fred Rolland, Allon Mureinik, Dan Kenigsberg,

I'd like you to do a code review.  Please visit

    https://gerrit.ovirt.org/40876

to review the following change.

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

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

Added a requirement for qemu-kvm-ev and qemu-img-ev on CentOS >= 7.

qemu-*-ev rpms are upstream version for qemu-*-rhev.
These supply features needed that are not available in the regular
qemu-*, for example 'Live Snapshot'.

Change-Id: Ibbec719863d7b12150574a60d071f38bcfa62398
Signed-off-by: Idan Shaby <[email protected]>
Signed-off-by: Fred Rolland <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/40670
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <[email protected]>
Reviewed-by: Allon Mureinik <[email protected]>
---
M vdsm.spec.in
1 file changed, 11 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/76/40876/1

diff --git a/vdsm.spec.in b/vdsm.spec.in
index 63a711a..dafa85d 100644
--- a/vdsm.spec.in
+++ b/vdsm.spec.in
@@ -263,17 +263,21 @@
 %endif
 
 %if !%{fedora_koji_build}
-%global rhev_suffix -rhev
-%endif
+%if 0%{?rhev_build}
+%global qemu_suffix -rhev
+%else
+%global qemu_suffix -ev
+%endif  # rhev_build
+%endif  # !%{fedora_koji_build}
 
 %if 0%{?rhel}
 %if 0%{?rhel} >= 7
-Requires: qemu-kvm%{?rhev_suffix}
-Requires: qemu-img%{?rhev_suffix}
+Requires: qemu-kvm%{?qemu_suffix} >= 10:1.5.3-60.el7
+Requires: qemu-img%{?qemu_suffix} >= 10:1.5.3-60.el7
 %else
-Requires: qemu-kvm%{?rhev_suffix} >= 2:0.12.1.2-2.415.el6_5.3
-Requires: qemu-img%{?rhev_suffix} >= 2:0.12.1.2-2.415.el6_5.3
-%endif
+Requires: qemu-kvm%{?qemu_suffix} >= 2:0.12.1.2-2.422
+Requires: qemu-img%{?qemu_suffix} >= 2:0.12.1.2-2.422
+%endif  # rhel > = 7
 %else
 Requires: qemu-kvm >= 2:0.15.0-4
 Requires: qemu-img >= 2:0.15.0-4


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibbec719863d7b12150574a60d071f38bcfa62398
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Freddy Rolland <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Fred Rolland <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to