copy volume to image/image to volume doesn't detach volume.

Bug #1194962 reported by Walt Boring
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Walt Boring

Bug Description

The iSCSI base driver's copy volume to image and copy image to volume code doesn't include a call
to detach a volume once the copying is done.

Tags: brick
Changed in cinder:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/32650
Committed: http://github.com/openstack/cinder/commit/0a155bea36aee44c5e7ecd026efe9537eee25eda
Submitter: Jenkins
Branch: master

commit 0a155bea36aee44c5e7ecd026efe9537eee25eda
Author: Walter A. Boring IV <email address hidden>
Date: Tue Jun 11 15:27:56 2013 -0700

    Add Brick iSCSI attach/detach.

    This patch adds the new brick initiator connector object
    which contains the code to do volume attach and detach to a host
    machine. It includes hooks to work in both cinder and nova.
    Nova has different exec wrapper and it also needs to talk to a
    hypervisor at certain points during detach.

    This patch also pulls the copy/pasted code from nova in the base
    ISCSIDriver's _attach_volume method to use this new brick library.

    This patch also includes a fix in the ISCSIDriver's
    copy_volume_to_image code that didn't actually detach a volume
    when it was done with the operation.
    Bug 1194962

    This patch also includes a fix for iSCSI detaches where the
    iSCSI LUN wasn't being removed from the system until the very
    last detach issues an iscsiadm logout.
    Bug 1112483

    This patch includes a fix for iSCSI multipath detaches where
    the multipath device and the iSCSI LUNs for the multipath device
    were never removed from the kernel.
    Bug 1112483

    Blueprint cinder-refactor-attach

    Change-Id: Ieb181f896adb9230bbb6a2e5c42f261d61a0f140

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: havana-2 → 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.