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

Merge "Set the `device` value in the attachment dict"

This commit is contained in:
Zuul 2018-06-26 14:33:12 +00:00 committed by Gerrit Code Review
parent ca2a0e4db6 297374e41b
commit 3bcccb7260
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
brick_cinderclient_ext/volume_actions.py
View File

@ -65,6 +65,12 @@ class ConnectVolume(VolumeAction):
mountpoint, mode, hostname):
device_info = brick_connector.connect_volume(connection_data)
# NOTE(flaper87): mountpoint doesn't seem to be used at all, we should
# perhaps consider removing it from the connect args. Unsure whether
# `path` exists for all the different drivers so, using the value of
# `mountpoint` as the default.
mountpoint = device_info.get('path', mountpoint)
self.volumes_client.volumes.attach(self.volume_id, instance_uuid=None,
mountpoint=mountpoint,
mode=mode,