Francesco Romani has posted comments on this change.

Change subject: vm: hyperv: support more libvirt optimizations
......................................................................


Patch Set 13:

(1 comment)

https://gerrit.ovirt.org/#/c/29234/13/vdsm.spec.in
File vdsm.spec.in:

Line 278: %endif
Line 279: 
Line 280: %if 0%{?rhel}
Line 281: %if 0%{?rhel} >= 7
Line 282: Requires: qemu-kvm%{?rhev_suffix} >= 1.5.3-60.el7
> If this works for you, I'll do cleanup in a followup patch.
update (still for followup patch): the rhev/no rhev split was intentionally 
added to work in a nicer way with fedora:

https://gerrit.ovirt.org/#/c/33367/4/vdsm.spec.in

will require more discussion, a simple patch to revert change 33367 seems not 
good enough.
Line 283: Requires: qemu-img%{?rhev_suffix} >= 1.5.3-60.el7
Line 284: %else
Line 285: Requires: qemu-kvm%{?rhev_suffix} >= 2:0.12.1.2-2.422
Line 286: Requires: qemu-img%{?rhev_suffix} >= 2:0.12.1.2-2.422


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia10cb4ce254b6e6f560c045aff9e68846fe4771f
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Martin Polednik <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to