openstack/cinder
Code Issues Proposed changes

VMAX: set faked max_oversubscription_ratio

Browse Source 
FakeConfiguration is used in VMAX tests. Without a valueset for
max_oversubscription_ratio, this results in test failures with the
error:

"TypeError: float() argument must be a string or a number, not 'NoneType'"

This adds a default value for this config option in the fake config.

Change-Id: Ia0b9772823fe9d093cfafe3ee241aad5b21fc3fd
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
This commit is contained in:
Sean McGinnis 2018-08-31 10:50:45 -05:00
parent ca9a8cf037
commit 157f6392a4
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
cinder/tests/unit/volume/drivers/dell_emc/vmax/test_vmax.py
View File

@ -1215,6 +1215,7 @@ class FakeConfiguration(object):
self.volume_backend_name = volume_backend_name self.volume_backend_name = volume_backend_name
self.config_group = volume_backend_name self.config_group = volume_backend_name
self.san_is_local = False self.san_is_local = False
self.max_over_subscription_ratio = 1
if replication_device: if replication_device:
self.replication_device = [replication_device] self.replication_device = [replication_device]
for key, value in kwargs.items(): for key, value in kwargs.items():