]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Declare multiattach is True in Dell Eqlx driver
authorwuyuting <wytdahu@gmail.com>
Wed, 25 Nov 2015 02:00:44 +0000 (10:00 +0800)
committerwuyuting <wytdahu@gmail.com>
Thu, 26 Nov 2015 01:10:38 +0000 (09:10 +0800)
An Eqlx volume can be attached to multi instances, however,
this feature is not declared in eqlx driver. As a result,
if a user create a multi-attached volume using eqlx as
backend, they will be failed.

In this commit, we declared multiattach is True in eqlx
driver.

Change-Id: Ia8e94e739c2573f1f26d8cb2713a6b851adf9428
Closes-Bug: #1519570

cinder/tests/unit/test_eqlx.py
cinder/volume/drivers/eqlx.py

index e81ed6d589f6ffb843ed35893a3b096cadb80895..f4e184b60d1910b1c47c29b30e8420c8cef24ced 100644 (file)
@@ -297,6 +297,7 @@ class DellEQLSanISCSIDriverTestCase(test.TestCase):
             self.assertEqual(not thin_enabled,
                              stats['thick_provisioning_support'])
             self.assertEqual('Dell', stats['vendor_name'])
+            self.assertTrue(stats['multiattach'])
 
     def test_get_space_in_gb(self):
         self.assertEqual(123.0, self.driver._get_space_in_gb('123.0GB'))
index 3e01f50f615c225fd6bb23a6e0831e963d29e222..aed59b6138badec6c9e1153582cd4830da2e18c2 100644 (file)
@@ -326,6 +326,7 @@ class DellEQLSanISCSIDriver(san.SanISCSIDriver):
 
         data['total_capacity_gb'] = 0
         data['free_capacity_gb'] = 0
+        data['multiattach'] = True
 
         provisioned_capacity = 0