]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix the iSER legacy usage in the new targets flow
authorAviram Bar-Haim <aviramb@mellanox.com>
Fri, 16 Jan 2015 00:45:30 +0000 (02:45 +0200)
committerAviram Bar-Haim <aviramb@mellanox.com>
Sun, 18 Jan 2015 11:25:14 +0000 (13:25 +0200)
commit009a6c301bb9f366eaaa76e1026ae962deda6d3b
tree8017bcffe05f1fe24dfca45f9121df98e468e6e9
parent42bda60d9bb63411f273f9c24fdf91ce6470696c
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
cinder/volume/driver.py
cinder/volume/targets/iser.py