diff --git a/manila/share/drivers/vastdata/driver.py b/manila/share/drivers/vastdata/driver.py index 5c3dda33c0..706a460306 100644 --- a/manila/share/drivers/vastdata/driver.py +++ b/manila/share/drivers/vastdata/driver.py @@ -169,7 +169,7 @@ class VASTShareDriver(driver.ShareDriver): return f"{root}/manila-{share_id}" def create_share(self, context, share, share_server=None): - return self._ensure_share(share)[0] + return self._ensure_share(share) def delete_share(self, context, share, share_server=None): """Called to delete a share""" diff --git a/manila/tests/share/drivers/vastdata/test_driver.py b/manila/tests/share/drivers/vastdata/test_driver.py index 4c43decc89..a307207c89 100644 --- a/manila/tests/share/drivers/vastdata/test_driver.py +++ b/manila/tests/share/drivers/vastdata/test_driver.py @@ -192,12 +192,18 @@ class VASTShareDriverTestCase(unittest.TestCase): mock_rest.views.ensure.assert_called_once_with( name="fakeid", path="/fake/manila-fakeid", policy_id=1 ) - self.assertDictEqual( + self.assertListEqual( location, - { - 'path': '1.1.1.0:/fake/manila-fakeid', - 'is_admin_only': False - } + [ + { + 'path': '1.1.1.0:/fake/manila-fakeid', + 'is_admin_only': False + }, + { + 'path': '1.1.1.1:/fake/manila-fakeid', + 'is_admin_only': False + }, + ] ) if not policy: mock_rest.views.update.assert_called_once_with( diff --git a/releasenotes/notes/bug-2088269-vast-add-multiple-export-locations-39243a9091c145f9.yaml b/releasenotes/notes/bug-2088269-vast-add-multiple-export-locations-39243a9091c145f9.yaml new file mode 100644 index 0000000000..033cbd70b8 --- /dev/null +++ b/releasenotes/notes/bug-2088269-vast-add-multiple-export-locations-39243a9091c145f9.yaml @@ -0,0 +1,8 @@ +--- +fixes: + - | + `Launchpad bug #2037422 `_: + Fixed an issue within the VASTData driver that made only one export path + available for mounting, regardless if more mount paths were available. Now, + the amount of export locations will correspond to the way the storage is + configured for better load balancing.