]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Add Multi-connection support to XIV
authorAlon Marx <alonma@il.ibm.com>
Sun, 19 Apr 2015 05:56:35 +0000 (08:56 +0300)
committerAlon Marx <alonma@il.ibm.com>
Tue, 28 Apr 2015 12:35:33 +0000 (15:35 +0300)
Add field in cinder.conf to hold a list of management IP addresses to connect
to the XIV backend.

DocImpact
Change-Id: Ie4eceb315188765747dc02a0a77b5a1d2c692fb5
Implements: blueprint xiv-multiple-mgmt-ips

cinder/tests/unit/test_ibm_xiv_ds8k.py
cinder/volume/drivers/ibm/xiv_ds8k.py

index b169505fbf00b4dcd3bd346c6513516fdfec88ff..5215fe16bf80c21dbcfdc484878ffb8bc68072a8 100644 (file)
@@ -275,6 +275,7 @@ class XIVDS8KVolumeDriverTest(test.TestCase):
         configuration.xiv_ds8k_connection_type = 'iscsi'
         configuration.xiv_chap = 'disabled'
         configuration.san_ip = FAKE
+        configuration.management_ips = FAKE
         configuration.san_login = FAKE
         configuration.san_clustername = FAKE
         configuration.san_password = FAKE
index 1d44eda64d577a771f21fead8cde19405beb3abd..a58eb830e854d18e2f8532a98859a69044a9b19e 100644 (file)
@@ -45,6 +45,10 @@ xiv_ds8k_opts = [
         choices=['disabled', 'enabled'],
         help='CHAP authentication mode, effective only for iscsi'
         ' (disabled|enabled)'),
+    cfg.StrOpt(
+        'management_ips',
+        default='',
+        help='List of Management IP addresses (separated by commas)'),
 ]
 
 CONF = cfg.CONF
@@ -78,7 +82,8 @@ class XIVDS8KDriver(san.SanDriver):
                 "xiv_ds8k_vol_pool": self.configuration.san_clustername,
                 "xiv_ds8k_connection_type":
                 self.configuration.xiv_ds8k_connection_type,
-                "xiv_chap": self.configuration.xiv_chap
+                "xiv_chap": self.configuration.xiv_chap,
+                "management_ips": self.configuration.management_ips
             },
             LOG,
             exception,