From 9ddce659a999754464d81b35d39cc5eace84a76c Mon Sep 17 00:00:00 2001 From: TommyLike Date: Fri, 1 Dec 2017 08:36:38 +0800 Subject: [PATCH] Remove 'end_version' parameter in backup update Remove unnecessary 'end_version' which was added in patch [1]. [1]: 2255fc99da9752737dcaa96ae4507b646074afb2 Change-Id: If014ba9c12bf662ef23ebf17decfa59273c63944 --- cinderclient/v3/shell.py | 2 +- cinderclient/v3/volume_backups.py | 5 +---- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/cinderclient/v3/shell.py b/cinderclient/v3/shell.py index db23d83c0..fc7ead144 100644 --- a/cinderclient/v3/shell.py +++ b/cinderclient/v3/shell.py @@ -906,7 +906,7 @@ def do_migrate(cs, args): six.text_type(e))) -@api_versions.wraps('3.9', '3.43') +@api_versions.wraps('3.9') @utils.arg('backup', metavar='', help='Name or ID of backup to rename.') @utils.arg('--name', nargs='?', metavar='', diff --git a/cinderclient/v3/volume_backups.py b/cinderclient/v3/volume_backups.py index 0ac9e1b93..555ace87f 100644 --- a/cinderclient/v3/volume_backups.py +++ b/cinderclient/v3/volume_backups.py @@ -26,15 +26,12 @@ VolumeBackup = volume_backups.VolumeBackup class VolumeBackupManager(volume_backups.VolumeBackupManager): - @api_versions.wraps("3.9", "3.43") + @api_versions.wraps("3.9") def update(self, backup, **kwargs): """Update the name or description for a backup. :param backup: The :class:`Backup` to update. """ - # NOTE(jdg): Placing 3.43 in versions.wraps above for clarity, - # but it's irrelevant as this just uses the kwargs, should we - # remove that? if not kwargs: return