From c7259c1792e4b9547cfa31ab69acb70b2ee4abf9 Mon Sep 17 00:00:00 2001 From: Sheel Rana Date: Wed, 25 May 2016 12:28:35 +0530 Subject: [PATCH] 'display-name' is not used in snapshot_manage Currently snapshot-manage operation uses name in request body. But handling for both name and display-name is present in code which can be removed. This is not an issue just minor update. Change-Id: I7a4d0c9caa344640aca173744a35a158f56c721e --- cinder/api/contrib/snapshot_manage.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/cinder/api/contrib/snapshot_manage.py b/cinder/api/contrib/snapshot_manage.py index e797dd27e6e..6befe377fae 100644 --- a/cinder/api/contrib/snapshot_manage.py +++ b/cinder/api/contrib/snapshot_manage.py @@ -113,10 +113,7 @@ class SnapshotManageController(wsgi.Controller): snapshot_parameters['metadata'] = snapshot.get('metadata', None) snapshot_parameters['description'] = snapshot.get('description', None) - # NOTE(wanghao) if name in request body, we are overriding the 'name' - snapshot_parameters['name'] = snapshot.get('name', - snapshot.get('display_name') - ) + snapshot_parameters['name'] = snapshot.get('name') try: new_snapshot = self.volume_api.manage_existing_snapshot(