openstack
/
cinder
Code Issues Proposed changes

Fixed invalid number of arguments bug in ScaleIO driver 

Change-Id: I6946f531e2bed4b3f7e4491ca26876dd68fd0fc8
Closes-Bug: #1789174
Signed-off-by: Matan Sabag <matan.sabag@emc.com>
This commit is contained in:
Matan Sabag 2018-08-27 08:57:27 +01:00
parent 1e65b764f6
commit 13a6689ccb
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cinder/tests/unit/volume/drivers/dell_emc/scaleio/mocks.py
View File

@ -40,7 +40,7 @@ class ScaleIODriver(driver.ScaleIODriver):
def unmanage(self, volume):
pass
def _is_volume_creation_safe(self, _pd, _sp, _pt):
def _is_volume_creation_safe(self, _pd, _sp):
return True

3
cinder/volume/drivers/dell_emc/scaleio/driver.py
View File

@ -600,8 +600,7 @@ class ScaleIODriver(driver.VolumeDriver):
provisioning = "ThickProvisioned"
allowed = self._is_volume_creation_safe(protection_domain_name,
storage_pool_name,
provisioning)
storage_pool_name)
if not allowed:
# Do not allow thick volume creation on this backend.
# Volumes may leak data between tenants.