openstack
/
cinder
Code Issues Proposed changes

PowerMax Driver - Snapshot restore payload fix 

Removing deviceNameListTarget from snapshot restore
payloads. Using this option at 90 will return a
500 error stating that it should not be specified.
This specifically affects calls against Unisphere
90 endpoint and not later PowerMAX Driver for
Cinder releases which use Unisphere's 91 endpoints
when reverting to snapshots.

Change-Id: Ibbe3e8671a22564828ecbd329b106d6d990e2d09
Closes-Bug: 1874697
This commit is contained in:
odonos12 2020-06-29 16:51:48 +01:00
parent 25ed65f1fa
commit 70ab1b359a
2 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
cinder/tests/unit/volume/drivers/dell_emc/powermax/test_powermax_rest.py
View File

@ -1066,7 +1066,6 @@ class PowerMaxRestTest(test.TestCase):
'symforce': 'false', 'nocopy': 'false',
'generation': 0}
payload_restore = {'deviceNameListSource': [{'name': source_id}],
'deviceNameListTarget': [{'name': source_id}],
'action': 'Restore',
'star': 'false', 'force': 'false'}
with mock.patch.object(

1
cinder/volume/drivers/dell_emc/powermax/rest.py
View File

@ -1791,7 +1791,6 @@ class PowerMaxRest(object):
if action == "Restore":
operation = 'Restore snapVx snapshot'
payload = {"deviceNameListSource": [{"name": source_id}],
"deviceNameListTarget": [{"name": source_id}],
"action": action,
"star": 'false', "force": 'false'}
elif action in ('Link', 'Unlink'):