Change in vdsm[master]: spec: updated qemu-*-rhev version

amureini at redhat.com amureini at redhat.com
Mon May 11 07:59:18 UTC 2015


Allon Mureinik has posted comments on this change.

Change subject: spec: updated qemu-*-rhev version
......................................................................


Patch Set 2: Code-Review-1

(3 comments)

https://gerrit.ovirt.org/#/c/40671/2//COMMIT_MSG
Commit Message:

Line 3: AuthorDate: 2015-05-11 09:18:38 +0300
Line 4: Commit:     Fred Rolland <frolland at redhat.com>
Line 5: CommitDate: 2015-05-11 09:18:38 +0300
Line 6: 
Line 7: spec: updated qemu-*-rhev version
This is true for all ELs, not only RHEV. How about "updated qemu-* requirements on EL"?
Line 8: 
Line 9: Updated the build version for qemu-kvm-rhev and qemu-img-rhev for a rhev
Line 10: build on rhel >= 7.
Line 11: 


Line 6: 
Line 7: spec: updated qemu-*-rhev version
Line 8: 
Line 9: Updated the build version for qemu-kvm-rhev and qemu-img-rhev for a rhev
Line 10: build on rhel >= 7.
Again - this isn't only RHEV, it's any EV.

Worth mentioning something like this too:
"""
The required qemu-img version solves a crash when using qemu-img convert -t none on some block storage devices (see https://bugzilla.redhat.com/show_bug.cgi?id=1203543 for details).
"""
Line 11: 
Line 12: Change-Id: Ifbd1fdd425f773776c5ce7596a0f339480a3296c
Line 13: Signed-off-by: Idan Shaby <ishaby at redhat.com>
Line 14: Signed-off-by: Fred Rolland <frolland at redhat.com>


https://gerrit.ovirt.org/#/c/40671/2/vdsm.spec.in
File vdsm.spec.in:

Line 284: 
Line 285: %if 0%{?rhel}
Line 286: %if 0%{?rhel} >= 7
Line 287: Requires: qemu-kvm%{?qemu_suffix} >= 10:1.5.3-60.el7_1.2
Line 288: Requires: qemu-img%{?qemu_suffix} >= 10:1.5.3-60.el7_1.2
Should be >= 10:2.1.2-23.el7_1.2
Line 289: %else
Line 290: Requires: qemu-kvm%{?qemu_suffix} >= 2:0.12.1.2-2.422
Line 291: Requires: qemu-img%{?qemu_suffix} >= 2:0.12.1.2-2.422
Line 292: %endif  # rhel > = 7


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifbd1fdd425f773776c5ce7596a0f339480a3296c
Gerrit-PatchSet: 2
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: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland 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