create export should be separated from volume create (LVM Driver)

Bug #1248947 reported by John Griffith
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Medium
John Griffith

Bug Description

Certain iSCSI Targets can't set up the target until they have information regarding the host/initiator that the volume will be attached to, but currently in the LVM driver we deal with the target creation and export on volume create.

Move the create export out of the create method.

Tags: drivers lvm
tags: added: drivers lvm
Changed in cinder:
status: New → In Progress
Changed in cinder:
milestone: icehouse-1 → icehouse-2
Thierry Carrez (ttx)
Changed in cinder:
milestone: icehouse-2 → icehouse-3
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/69936

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/70291
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=beecd769af02ba5915be827d28a7b46d970e41b0
Submitter: Jenkins
Branch: master

commit beecd769af02ba5915be827d28a7b46d970e41b0
Author: John Griffith <email address hidden>
Date: Fri Jan 31 03:53:37 2014 +0000

    Remove create_export from volume create

    Currently the LVM driver creates iscsi targets on
    volume creation, even though it's only used at
    attach time. This isn't necessary and in fact
    causes issues if a volume is extended because the
    target information isn't currently updated after the
    extend. In addition a number of other drivers inclduing
    the LIO iscsi driver require the target be created with
    initiator info, so the tgt that's created initially
    again isn't valid.

    This change removes the create_export call from the
    volume create process and makes it part of the
    managers intialize_connection routine which is
    more appropriate.

    This change also removes the target on detach since
    it's not used any longer and again as the volume may
    be modified or extended.

    Change-Id: I0b7dfdaf7e49a069da22f22f459861b0b49430a4
    Closes-Bug: 1248947

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: icehouse-3 → 2014.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.