openstack/placement
Code Issues Proposed changes

Pass mountpoint to volume attachment_create with connector

Browse Source 
Similar to I11ba269c3f7a2e7707b2b7e27d26eb7a2c948a82, when
we create a volume attachment with a connector, we need to
also provide the mountpoint via the connector to Cinder,
because internally within Cinder the attachment_create code
is calling attachment_update if a connector is provided.

Change-Id: If3afe8d8bd6b8c327ccc7d1140053bccaf7e1ad7
Closes-Bug: #1737779
This commit is contained in:
Matt Riedemann 2017-12-12 12:43:53 -05:00
parent 0a00a675ea
commit 89eb6a9274
1 changed files with 1 additions and 1 deletions

2
nova/tests/fixtures.py

@ -1575,7 +1575,7 @@ class CinderFixtureNewAttachFlow(fixtures.Fixture):
return {'save_volume_id': new_volume_id}
def fake_attachment_create(_self, context, volume_id, instance_uuid,
connector=None):
connector=None, mountpoint=None):
attachment_id = uuidutils.generate_uuid()
if self.attachment_error_id is not None:
attachment_id = self.attachment_error_id