diff --git a/cinder/tests/test_coraid.py b/cinder/tests/test_coraid.py index 63edb9581bd..7ee9573bee2 100644 --- a/cinder/tests/test_coraid.py +++ b/cinder/tests/test_coraid.py @@ -489,7 +489,8 @@ class CoraidDriverIntegrationalTestCase(CoraidDriverLoginSuccessTestCase): delete_snapshot_request = {'addr': 'cms', 'data': { 'repoName': fake_repository_name, - 'lvName': fake_snapshot_name}, + 'lvName': fake_snapshot_name, + 'newLvName': 'noop'}, 'op': 'orchStrLunMods', 'args': 'delClSnap'} pack_data(delete_snapshot_request) diff --git a/cinder/volume/drivers/coraid.py b/cinder/volume/drivers/coraid.py index d97b2100039..652ae43a95f 100644 --- a/cinder/volume/drivers/coraid.py +++ b/cinder/volume/drivers/coraid.py @@ -353,7 +353,12 @@ class CoraidAppliance(object): request = {'addr': 'cms', 'data': { 'repoName': repository_name, - 'lvName': snapshot_name}, + 'lvName': snapshot_name, + # NOTE(novel): technically, the 'newLvName' is not + # required for 'delClSnap' command. However, some + # versions of ESM have a bug that fails validation + # if we don't specify that. Hence, this fake value. + 'newLvName': "noop"}, 'op': 'orchStrLunMods', 'args': 'delClSnap'} esm_result = self.esm_command(request)