]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Fix ISCSIDriver initialized connection volume type
authorAviram Bar-Haim <aviramb@mellanox.com>
Thu, 2 Apr 2015 14:37:04 +0000 (17:37 +0300)
committerAviram Bar-Haim <aviramb@mellanox.com>
Thu, 2 Apr 2015 15:42:50 +0000 (18:42 +0300)
ISCSIDriver initialize connection is a duplicated function
left in kilo to serve drivers that don't use the new targets model.

In Kilo there is a new parameter of iscsi_protocol, that is set to
iscsi by default and can be changed to iser in order to enable RDMA.

In order support RDMA in drivers that still inherit from ISCSIDriver,
driver_volume_type should be set to iscsi_protocol parameter value
instead of a hard coded value until this function duplication will be solved.

Closes-Bug: #1438833

Change-Id: If51ff933b75bf475c779d2599776f8a83acc08e5

cinder/volume/driver.py

index 474d80cec8dd271c1f4edc0fcd3f6582ad285455..5e4a70ce1c87cfcae800b2c99ec426396f2f7b14 100644 (file)
@@ -1572,7 +1572,8 @@ class ISCSIDriver(VolumeDriver):
         # this base class and use this init data
         iscsi_properties = self._get_iscsi_properties(volume)
         return {
-            'driver_volume_type': 'iscsi',
+            'driver_volume_type':
+                self.configuration.safe_get('iscsi_protocol'),
             'data': iscsi_properties
         }