Bug 1215744 - Rebuild qemu-img-ev and qemu-kvm-ev to include fix for "Bug 1203543 - bdrv_make_zero() passes a too large nb_sectors value to bdrv_write_zeroes()"
Summary: Rebuild qemu-img-ev and qemu-kvm-ev to include fix for "Bug 1203543 - bdrv_ma...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: vdsm
Version: 3.5
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: 3.5.3
Assignee: Sandro Bonazzola
QA Contact: Ori Gofen
URL:
Whiteboard: external
Depends On: 1203543 1219754
Blocks: 1209034 1216911
TreeView+ depends on / blocked
 
Reported: 2015-04-27 16:14 UTC by Allon Mureinik
Modified: 2016-05-26 01:49 UTC (History)
13 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
see https://bugzilla.redhat.com/show_bug.cgi?id=1203543 Doc Text
Clone Of:
Environment:
Last Closed: 2015-06-15 08:38:50 UTC
oVirt Team: External
Embargoed:
sbonazzo: ovirt_requires_release_note?


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 40687 0 master MERGED build: qemu-kvm-ev-2.1.2-23.el7_1.2 Never

Description Allon Mureinik 2015-04-27 16:14:45 UTC
Description of problem:
Bug 1203543 - "bdrv_make_zero() passes a too large nb_sectors value to bdrv_write_zeroes()" describes a regression that prevents oVirt from creating a template from a preallocated disc on block storage with some storage servers (the issue was noticed on XtremeIO, we do not currently have data on other storage vendors/drivers that present this problem).

Once it is fixed (presumably in qemu-kvm-rhev-2.1.2-23.el7_1_1.2, according to bug 1203543), we need to respin the -ev packages so this fix can be consumed by oVirt on EL.

Version-Release number of selected component (if applicable):
oVirt 3.5.0 and above

How reproducible:
100% consistent on storage servers known to have this issue (currently, only XtremeIO).

Comment 1 Allon Mureinik 2015-04-27 16:29:23 UTC
Sandro, as discussed on bug 1209034, once bug 1203543 is fixed, we need this fix repackaged asap, even as an async release.

I'm not sure what component and target release this should be flagged for, so feel free to change it.

Thanks!

Comment 2 Eyal Edri 2015-04-28 11:23:32 UTC
moving to 3.5.4 due to capacity planning for 3.5.3.
if you believe this should remain in 3.5.3, please sync with pm/dev/qe and a full triple ack for it. also - ensure priority is set accordingly to the bug status.

Comment 3 Sandro Bonazzola 2015-04-29 07:54:11 UTC
Re-targeting to 3.5.2 as this must be released async.

Comment 4 Sandro Bonazzola 2015-05-08 07:12:43 UTC
Dan, this requires a backport of https://gerrit.ovirt.org/40648, causing an async release of vdsm for 3.5.2 too.
Please ack for delivering this in 3.5.2 instead of 3.5.3

Comment 5 Sandro Bonazzola 2015-05-08 07:59:34 UTC
Published in ovirt-master-snapshot repo, pending needinfo for backporting to ovirt-3.5-snapshot and ovirt-3.5 repositories.

Comment 6 Dan Kenigsberg 2015-05-08 08:57:26 UTC
(In reply to Sandro Bonazzola from comment #4)
> Dan, this requires a backport of https://gerrit.ovirt.org/40648, causing an
> async release of vdsm for 3.5.2 too.
> Please ack for delivering this in 3.5.2 instead of 3.5.3

Why is this required? Because you don't want to include this fix in qemu-kvm-rhev?

In any case, I do not mind creating an ovirt-3.5.2 branch and including that caps.py change to it. I've created https://gerrit.ovirt.org/gitweb?p=vdsm.git;a=shortlog;h=refs%2Fheads%2Fovirt-3.5.2 on top of a fairly-important fix to a memory leak in el6.

Comment 7 Sandro Bonazzola 2015-05-08 11:17:20 UTC
In 3.5.2 we're delivering qemu-img-rhev-1.5.3-86.el7.1.
The request here is to bump to qemu-kvm-common-ev-2.1.2-23.el7_1.2

We don't have a -rhev package for 2.*, only -ev ones.

Comment 8 Allon Mureinik 2015-05-10 06:58:48 UTC
(In reply to Dan Kenigsberg from comment #6)
> (In reply to Sandro Bonazzola from comment #4)
> > Dan, this requires a backport of https://gerrit.ovirt.org/40648, causing an
> > async release of vdsm for 3.5.2 too.
> > Please ack for delivering this in 3.5.2 instead of 3.5.3
> 
> Why is this required? Because you don't want to include this fix in
> qemu-kvm-rhev?
Because we don't want to have upstream EL builds relying on Red Hat (-RHev) branded rpms.

> 
> In any case, I do not mind creating an ovirt-3.5.2 branch and including that
> caps.py change to it. I've created
> https://gerrit.ovirt.org/gitweb?p=vdsm.git;a=shortlog;h=refs%2Fheads%2Fovirt-
> 3.5.2 on top of a fairly-important fix to a memory leak in el6.
Nice one, thanks!

Comment 9 Sandro Bonazzola 2015-05-11 12:51:03 UTC
Re-targeted to 3.5.3.

Comment 10 Ori Gofen 2015-06-02 09:22:29 UTC
Verified on vt15.1
qemu version: qemu-kvm-rhev-2.1.2-23.el7_1.3.x86_64 includes the fix.

for more info: bz #1220333 bz #1221192

Comment 11 Sandro Bonazzola 2015-06-15 08:38:50 UTC
This is an automated message.
oVirt 3.5.3 has been released on June 15th 2015 and should include the fix for this BZ. Moving to closed current release.


Note You need to log in before you can comment on or make changes to this bug.