]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix FC connection handling in the storwize driver
authorZoltan Arnold Nagy <nag@zurich.ibm.com>
Wed, 12 Feb 2014 22:01:10 +0000 (23:01 +0100)
committerLi Min Liu <liminliu@cn.ibm.com>
Mon, 24 Feb 2014 02:26:45 +0000 (10:26 +0800)
commit182e071a113fef7138a77bfae0d3b432e875d289
treef11726bf82319e432e1876e2fba6486a875286db
parentf028c8ac509acac69f29323fd9fefea143693a75
Fix FC connection handling in the storwize driver

Creating FC connections was not working due to several issues:

1. The lsfabric command in the Storwize/SVC omitted the delimeter and so
the output was not correctly parsed by the driver.
2. In some cases comparisons of WWPNs were not case insensitive - ensure
we compare lower-case WWPNs.
3. If the host supplied extra information in the connector (e.g. iSCSI
initiator name in case of an FC setup, or FC WWNNs/WWPNs in an iSCSI
setup), the driver would try to make those host mappings on the storage
too.  This could lead into unwanted behaviour and confusing error
messages (for example, in an FC setup reaching the maximum number of
iSCSI mappings).

In addition, unit tests weren't properly testing FC - instead they were
always testing iSCSI due to an error in parsing extra_specs. This patch
fixes that error and the unit tests as well.

These changes must all be made together for unit tests to pass.

Closes-bug: 1279758
Change-Id: I64e21609ba089cf5bfd52ce2644f6f229bcc69dc
cinder/tests/test_storwize_svc.py
cinder/volume/drivers/ibm/storwize_svc/__init__.py
cinder/volume/drivers/ibm/storwize_svc/helpers.py
cinder/volume/drivers/ibm/storwize_svc/ssh.py