GPFS Raw file is copied by qemu-img even though copy-on-write option is specified

Bug #1201676 reported by Dinesh Subhraveti
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Bill Owen

Bug Description

When an image is fetched from Glance and stored as a volume, the image bits are unnecessarily copied during format conversion by qemu-img when source and target formats are both raw.

Thanks to Avishay for pointing out the error.

Tags: drivers gpfs
Changed in cinder:
assignee: nobody → Dinesh Subhraveti (subhraveti)
Revision history for this message
Avishay Traeger (avishay-il) wrote :

Note that this will be fixed in the GPFS driver, not in the generic version, and therefore is not a duplicate bug.

Revision history for this message
Avishay Traeger (avishay-il) wrote :

So some more details to avoid confusion with similarly-named bugs:

In the GPFS driver implementation of copy-image-to-volume, the image is on the GPFS file system and the driver can test to see if it's raw ahead of time, and simply symlink the file if it's already raw, rather than calling 'qemu-img convert' which copies data. This saves space and copy time.

tags: added: drivers gpfs
summary: - Raw file is copied by qemu-img even though copy-on-write option is
+ GPFS Raw file is copied by qemu-img even though copy-on-write option is
specified
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/38355

Changed in cinder:
assignee: Dinesh Subhraveti (subhraveti) → Bill Owen (billowen)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/38355
Committed: http://github.com/openstack/cinder/commit/84ca4b3698ae8512f6ba4cd72a57f662e35b59a0
Submitter: Jenkins
Branch: master

commit 84ca4b3698ae8512f6ba4cd72a57f662e35b59a0
Author: Bill Owen <email address hidden>
Date: Sun Jul 21 17:35:50 2013 -0700

    GPFS convert glance image to raw only when needed

    Modify _gpfs_fetch_to_raw to check image format. If already in raw
    format, no need to call convert_image. In that case, simply rename
    to destination name.

    Change-Id: I197d99194f93c02ce3c501de25433d89b23e6a35
    Fixes: bug #1201676

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → havana-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: havana-3 → 2013.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.