openstack/python-brick-cinderclient-ext
Code Issues Proposed changes

Fix to allow multipathing with attach/detach

Browse Source 
We were not piping the flags all the way through to
the os-brick calls...

Change-Id: I4b27bab4c358b2f0f1bb5c0240f9a8e736e12b10
This commit is contained in:
Patrick East
2017-06-30 09:54:29 -07:00
parent 8ad341db04
commit bb3593ed12
1 changed files with 11 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
brick_cinderclient_ext/client.py
View File

@@ -137,7 +137,9 @@ class Client(object):
# We can use the new attach/detach workflow
connector_properties = self.get_connector(
multipath=multipath,
enforce_multipath=enforce_multipath, nic=nic)
enforce_multipath=enforce_multipath,
nic=nic
)
instance_id = uuidutils.generate_uuid()
@@ -149,7 +151,10 @@ class Client(object):
cmd.verify(connection['driver_volume_type'])
brick_connector = self._brick_get_connector(
connection['driver_volume_type'], do_local_attach=True)
connection['driver_volume_type'],
do_local_attach=True,
use_multipath=multipath,
)
device_info = brick_connector.connect_volume(connection)
return device_info
@@ -181,7 +186,10 @@ class Client(object):
nic)
brick_connector = self._brick_get_connector(
connection['driver_volume_type'], do_local_attach=True)
connection['driver_volume_type'],
do_local_attach=True,
use_multipath=multipath,
)
with actions.DisconnectVolume(self.volumes_client, volume_id) as cmd:
cmd.disconnect(brick_connector, connection['data'], device_info)