qemu-img convert intermittently corrupts output images

[Dmitry Mescheryakov]

Upstream bug https://bugs.launchpad.net/qemu/+bug/1368815

MOS was set as affected by Vladimir Kuklin, ping him if you need details

[Alexei Sheplyakov (asheplyakov)]

In fact the up-to-date qemu versions (>= 2.1) rarely use FIEMAP (see the commit 4f11aa8a4035,
and the explanation in the commit c4875e5b2216 for more details).

Recently the FIEMAP related code has been removed (commit c4875e5b2216c). I think we should
get rid of FIEMAP too. I'm working on backporting the upstream changes.

[OSCI Robot (oscirobot)]

DEB package qemu has been built for project packages/precise/qemu
Package version == , package release ==

Changeset: https://review.fuel-infra.org/1348
project: packages/precise/qemu
branch: 6.0
author: Alexei Sheplyakov
committer: Alexei Sheplyakov
subject: Latest version from Ubuntu 14.04 and qemu-img fix
status: patchset-created

Files placed on repository:
qemu-common_2.0.0+dfsg-2ubuntu1.9_all.deb
qemu-guest-agent_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-keymaps_2.0.0+dfsg-2ubuntu1.9_all.deb
qemu-kvm_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-arm_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-common_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-mips_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-misc_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-ppc_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-sparc_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-x86_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-user-static_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-user_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-utils_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu_2.0.0+dfsg-2ubuntu1.9_amd64.deb

NOTE: Changeset is not merged, created temporary package repository.
DEB repository URL: /ubuntu-fuel-6.0-stable-1348/ubuntu

[Michael Semenov (msemenov)]

6.1:
According to https://bugs.launchpad.net/fuel/+bug/1383641 we will use Ubuntu 14.04(trusty) in MOS 6.1. New qemu and qemu-img versions will be used. So, most probably, this fix doesn't needed in 6.1, and bug should be moved into 'Won't fix' state. But let's don't do it right now because of 2 risks:
1. We are fail with Ubuntu 14.04 update for some reason.
2. There is no upstream fix with FIEMAP-free code (at the time #1383641 is fixed).

[OSCI Robot (oscirobot)]

DEB package qemu has been built for project packages/precise/qemu
Package version == , package release ==

Changeset: https://review.fuel-infra.org/1348
project: packages/precise/qemu
branch: 6.0
author: Alexei Sheplyakov
committer: Alexei Sheplyakov
subject: Latest version from Ubuntu 14.04 and qemu-img fix
status: change-merged

Files placed on repository:
qemu-common_2.0.0+dfsg-2ubuntu1.9_all.deb
qemu-guest-agent_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-keymaps_2.0.0+dfsg-2ubuntu1.9_all.deb
qemu-kvm_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-arm_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-common_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-mips_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-misc_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-ppc_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-sparc_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system-x86_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-system_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-user-static_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-user_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu-utils_2.0.0+dfsg-2ubuntu1.9_amd64.deb
qemu_2.0.0+dfsg-2ubuntu1.9_amd64.deb

Changeset merged. Package placed on primary repository
DEB repository URL: /ubuntu-fuel-6.0-stable/ubuntu

[Alexei Sheplyakov (asheplyakov)]

The bug has been fixed (more than 4 months ago) in qemu 2.0.0+dfsg-2ubuntu1.8
(the current version is 2.0.0+dfsg-2ubuntu1.10):

qemu (2.0.0+dfsg-2ubuntu1.8) trusty-proposed; urgency=medium

  * debian/qemu-system-x86.qemu-kvm.upstart: create /dev/kvm in a
    container. (LP: #1370199)
  * Cherrypick upstream patch to fix intermittent qemu-img corruption
    (LP: #1368815)
    - 501-block-raw-posix-fix-disk-corruption-in-try-fiemap
    - (note - 502-block-raw-posic-use-seek-hole-ahead-of-fiemap (which was
      also needed in utopic) appears to be unneeded here as the code being
      changed has not yet been switched to using try_fiemap)

 -- Serge Hallyn <email address hidden> Thu, 20 Nov 2014 11:24:51 -0600

[Alexei Sheplyakov (asheplyakov)]

Besides, assigning the Critical priorty to a bug which haven't been reproduced looks quite strange