iSER is broken after the transition of LVM drivers to use target objects

Bug #1411470 reported by Aviram Bar-Haim
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Medium
Aviram Bar-Haim

Bug Description

The iSER driver flow, after setting volume_driver to cinder.volume.drivers.lvm.LVMISERDriver, is broken in the new driver - target model in the master, due to a mismatch in the target prefixes and a lack of iSER target initialization with the correct parameters.

Reproduced in devstack with Cinder Master branch over an RDMA environment.

Changed in cinder:
assignee: nobody → Aviram Bar-Haim (aviramb)
Mike Perez (thingee)
Changed in cinder:
milestone: none → kilo-2
importance: Undecided → Medium
status: New → Confirmed
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/148097

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

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

commit 009a6c301bb9f366eaaa76e1026ae962deda6d3b
Author: Aviram Bar-Haim <email address hidden>
Date: Fri Jan 16 02:45:30 2015 +0200

    Fix the iSER legacy usage in the new targets flow

    The iSER driver flow, after setting volume_driver to LVMISERDriver,
    is broken in the new driver - target model in the master,
    due to a mismatch in the target prefixes and a lack of iSER target
    initialization, with the correct parameters.

    In order to fix this:
      1. The function initialize_connection, that sets the iSER required
         parameters, was added to cinder/volume/targets/iser.py (and overrides the
         new connection initialization in cinder/volume/targets/tgt.py).
      2. The iSER and iSCSI prefixes equalized, due to the removal of brick/iscsi
         with the different default prefix for ISERTgtAdm (and for stability).

    Closes-Bug:#1411470

    Change-Id: Ie4b9b85b47e9da20a1de1e10415766413cd01f04

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: kilo-2 → 2015.1.0
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.