iSCSI Drivers fall back to LVM for volume<->image

Bug #1090169 reported by Sam Morrison
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Medium
Sam Morrison

Bug Description

It falls back to trying to use an LVM drive and fails. Should at least raise a NotImplemented error until this is supported.

Revision history for this message
John Griffith (john-griffith) wrote :

Yup, the 3'rd party back-ends should raise for this until we implement something hire up in the stack to deal with it properly.

Changed in cinder:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
John Griffith (john-griffith) wrote :

Note, we should go through all of the drivers and add the over-riding call to ensure it doesn't grab the base classes (LVM) default.

tags: added: low-hanging-fruit
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/18108

Changed in cinder:
assignee: nobody → Sam Morrison (sorrison)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/18108
Committed: http://github.com/openstack/cinder/commit/403142a43f553b1488eae795c6dfff3967825bf1
Submitter: Jenkins
Branch: master

commit 403142a43f553b1488eae795c6dfff3967825bf1
Author: Sam Morrison <email address hidden>
Date: Fri Dec 14 13:49:46 2012 +1100

    Raise NotImplemented for drivers that don't support images

    Fixes bug #1090169

    Raise NotImplementedError on copy_image_to_volume and
    copy_volume_to_image on drivers that inherit from
    driver.ISCSIDriver that don't support this feature yet.

    Change-Id: I516d424a75dd21260f2ccc68467a8f6e6ac08e4d

Changed in cinder:
status: In Progress → Fix Committed
Sam Morrison (sorrison)
summary: - NetApp: driver doesn't support creating volume from image
+ iSCSI Drivers fall back to LVM for volume<->image
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → grizzly-2
status: Fix Committed → Fix Released
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/21195

Thierry Carrez (ttx)
Changed in cinder:
milestone: grizzly-2 → 2013.1
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.