Bug 1215744

Summary: 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()"
Product: [Retired] oVirt Reporter: Allon Mureinik <amureini>
Component: vdsmAssignee: Sandro Bonazzola <sbonazzo>
Status: CLOSED CURRENTRELEASE QA Contact: Ori Gofen <ogofen>
Severity: high Docs Contact:
Priority: high    
Version: 3.5CC: acanan, amureini, bazulay, bugs, danken, ecohen, gklein, lsurette, mgoldboi, rbalakri, sgotliv, yeylon, ylavi
Target Milestone: ---Keywords: Regression
Target Release: 3.5.3Flags: sbonazzo: ovirt_requires_release_note?
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: external
Fixed In Version: Doc Type: Bug Fix
Doc Text:
see https://bugzilla.redhat.com/show_bug.cgi?id=1203543 Doc Text
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-06-15 08:38:50 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: External RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1203543, 1219754    
Bug Blocks: 1209034, 1216911    

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.