HPE Lefthand: fix volume manage operation

If manage operation needs retype of a volume then
send proper arguments to retype function for
successful retype.

Change-Id: Idc8a1d51c9012bc6e67f60eb5629d0336b009076
Closes-Bug: #1710098
This commit is contained in:
kushal 2017-09-01 04:03:06 -07:00
parent 3042ec126a
commit d3f2055a6c
2 changed files with 16 additions and 2 deletions

View File

@ -2168,6 +2168,14 @@ class TestHPELeftHandISCSIDriver(HPELeftHandBaseDriver, test.TestCase):
self.driver_startup_call_stack + [
mock.call.modifyVolume(self.volume_id,
{'name': 'volume-12345'}),
mock.call.logout()],
self.driver_startup_call_stack + [
mock.call.getVolumeByName(self.volume_name),
mock.call.modifyVolume(self.volume_id,
{'isThinProvisioned': True,
'dataProtectionLevel': 0,
'isAdaptiveOptimizationEnabled':
True}),
mock.call.logout()])
self.assertEqual(expected_obj, obj)

View File

@ -172,9 +172,11 @@ class HPELeftHandISCSIDriver(driver.ISCSIDriver):
2.0.13 - Fix cloning operation related to provisioning, bug #1688243
2.0.14 - Fixed bug #1710072, Volume doesn't show expected parameters
after Retype
2.0.15 - Fixed bug #1710098, Managed volume, does not pick up the extra
specs/capabilities of the selected volume type.
"""
VERSION = "2.0.14"
VERSION = "2.0.15"
CI_WIKI_NAME = "HPE_Storage_CI"
@ -1255,11 +1257,15 @@ class HPELeftHandISCSIDriver(driver.ISCSIDriver):
LOG.info("Virtual volume %(disp)s '%(new)s' is being retyped.",
{'disp': display_name, 'new': new_vol_name})
# Creates a diff as it needed for retype operation.
diff = {}
diff['extra_specs'] = {key: (None, value) for key, value
in volume_type['extra_specs'].items()}
try:
self.retype(None,
volume,
volume_type,
volume_type['extra_specs'],
diff,
volume['host'])
LOG.info("Virtual volume %(disp)s successfully retyped to "
"%(new_type)s.",