Fixed positional order of arguments

The positional arguments of backup-create and configuration-attach
were different from all other commands.  'Instance' is now the
first argument for these commands as well.

DocImpact: This potentially affects any user scripts that may
exist.  Any instructions like:
'trove backup-create mybackup myinstance'
'trove configuration-attach myconfig myinstance'
should be changed to:
'trove backup-create myinstance mybackup.'
'trove configuration-attach myinstance myconfig'

Change-Id: I2ff93a1a6e26f3e33a60cbf3d9ad5ee074307e8f
Closes-Bug: #1281878
This commit is contained in:
Peter Stachowski
2014-05-06 12:42:48 -04:00
parent d02764e597
commit 30c924e343

View File

@@ -292,10 +292,10 @@ def do_backup_show(cs, args):
_print_instance(backup)
@utils.arg('instance', metavar='<instance>', help='ID of the instance.')
@utils.arg('--limit', metavar='<limit>',
default=None,
help='Return up to N number of the most recent backups.')
@utils.arg('instance', metavar='<instance>', help='ID of the instance.')
@utils.service_type('database')
def do_backup_list_instance(cs, args):
"""Lists available backups for an instance."""
@@ -332,8 +332,8 @@ def do_backup_delete(cs, args):
cs.backups.delete(args.backup)
@utils.arg('name', metavar='<name>', help='Name of the backup.')
@utils.arg('instance', metavar='<instance>', help='ID of the instance.')
@utils.arg('name', metavar='<name>', help='Name of the backup.')
@utils.arg('--description', metavar='<description>',
default=None,
help='An optional description for the backup.')
@@ -638,14 +638,14 @@ def do_datastore_version_show(cs, args):
# configuration group related functions
@utils.arg('configuration',
metavar='<configuration>',
type=str,
help='ID of the configuration group to attach to the instance.')
@utils.arg('instance',
metavar='<instance>',
type=str,
help='ID of the instance.')
@utils.arg('configuration',
metavar='<configuration>',
type=str,
help='ID of the configuration group to attach to the instance.')
@utils.service_type('database')
def do_configuration_attach(cs, args):
"""Attaches a configuration group to an instance."""