]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Deprecate *_multipath_enabled flag for IBM drivers
authorRyan McNair <rdmcnair@us.ibm.com>
Thu, 12 Nov 2015 15:06:32 +0000 (15:06 +0000)
committerRyan McNair <rdmcnair@us.ibm.com>
Fri, 20 Nov 2015 17:36:08 +0000 (17:36 +0000)
commit11edf336592b779cd05b1e3b44abc9c8c5a3a205
treee439be0366e527e68a4a89f3dd25b808aa44b613
parent02b68fde2615b5bb8b558902c02a3a1259056f83
Deprecate *_multipath_enabled flag for IBM drivers

Deprecate the *_multipath_enabled from the FlashSystem and
Storwize/SVC driver. When enabled, this flag was intended to create
a single mapping between the compute node and the storage device for
FC connections. However, all available storage WWNNs were still being
mapped to the compute node, and instead enabling this flag was
causing a terminate_connection to not cleanup all LUN mappings
(because Nova did not know of all the target_wwns that were actually
mapped). This can be seen by enabling the flag, connecting a volume,
disconnecting the volume and observing the stray LUN mappings under
'/dev/disk/by-path'.

Currently we do not have a way to limit the storage WWNNs which are
mapped during a FC connection, so we will ignore the config to fix
this broken behavior, and then remove the config next release.

DocImpact

Change-Id: I0019c1e0c0dbb0cd48a7f389376480e7fa1b1e07
Closes-Bug: #1516778
cinder/tests/unit/test_ibm_flashsystem.py
cinder/tests/unit/test_ibm_flashsystem_iscsi.py
cinder/tests/unit/test_storwize_svc.py
cinder/volume/drivers/ibm/flashsystem_common.py
cinder/volume/drivers/ibm/flashsystem_fc.py
cinder/volume/drivers/ibm/storwize_svc/__init__.py
cinder/volume/drivers/ibm/storwize_svc/storwize_svc_common.py