openstack/python-cinderclient
Code Issues Proposed changes

Remove 'end_version' parameter in backup update

Browse Source 
Remove unnecessary 'end_version' which was
added in patch [1].

[1]: 2255fc99da

Change-Id: If014ba9c12bf662ef23ebf17decfa59273c63944
This commit is contained in:
TommyLike
2017-12-01 08:36:38 +08:00
parent d17ae43e1d
commit 9ddce659a9
2 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
cinderclient/v3/shell.py
View File

@@ -906,7 +906,7 @@ def do_migrate(cs, args):
six.text_type(e))) six.text_type(e)))
@api_versions.wraps('3.9', '3.43') @api_versions.wraps('3.9')
@utils.arg('backup', metavar='<backup>', @utils.arg('backup', metavar='<backup>',
help='Name or ID of backup to rename.') help='Name or ID of backup to rename.')
@utils.arg('--name', nargs='?', metavar='<name>', @utils.arg('--name', nargs='?', metavar='<name>',

5
cinderclient/v3/volume_backups.py
View File

@@ -26,15 +26,12 @@ VolumeBackup = volume_backups.VolumeBackup
class VolumeBackupManager(volume_backups.VolumeBackupManager): class VolumeBackupManager(volume_backups.VolumeBackupManager):
@api_versions.wraps("3.9", "3.43") @api_versions.wraps("3.9")
def update(self, backup, **kwargs): def update(self, backup, **kwargs):
"""Update the name or description for a backup. """Update the name or description for a backup.
:param backup: The :class:`Backup` to update. :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: if not kwargs:
return return