diff options
author | Cedric Brandily <zzelle@gmail.com> | 2016-10-11 22:01:16 +0200 |
---|---|---|
committer | Christopher Brown <snecklifter@gmail.com> | 2017-07-05 14:04:56 +0000 |
commit | 537b085c8e3442e85058f0398f2a578d2d414bac (patch) | |
tree | 74150cb6f84903cc0e5d325e92c83ca0f2567b30 | |
parent | f4764d6ebe28c6b6040d29d180c5c71968b1ab10 (diff) | |
download | python-openstackclient-stable/newton.tar.gz |
Fix --shared/block-migration options in server migrate commandnewton-eolstable/newton
Currently, --shared-migration and --block-migration options effects are
reversed: --block-migration requests a migration with share,
--shared-migration a block-migration.
This change corrects OSC implementation and clarifies arguments passed
to novaclient (the root cause of the bug).
Change-Id: Ib682cff0c44d3b1304670f8606907b1762d8b1e7
Closes-Bug: #1518059
(cherry picked from commit 694a24c3093f5b2595b63ccf988da7972e532084)
-rw-r--r-- | openstackclient/compute/v2/server.py | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/openstackclient/compute/v2/server.py b/openstackclient/compute/v2/server.py index b655343f..9c7d14cc 100644 --- a/openstackclient/compute/v2/server.py +++ b/openstackclient/compute/v2/server.py @@ -975,15 +975,15 @@ class MigrateServer(command.Command): migration_group = parser.add_mutually_exclusive_group() migration_group.add_argument( '--shared-migration', - dest='shared_migration', - action='store_true', - default=True, + dest='block_migration', + action='store_false', + default=False, help=_('Perform a shared live migration (default)'), ) migration_group.add_argument( '--block-migration', - dest='shared_migration', - action='store_false', + dest='block_migration', + action='store_true', help=_('Perform a block live migration'), ) disk_group = parser.add_mutually_exclusive_group() @@ -1018,9 +1018,9 @@ class MigrateServer(command.Command): ) if parsed_args.live: server.live_migrate( - parsed_args.live, - parsed_args.shared_migration, - parsed_args.disk_overcommit, + host=parsed_args.live, + block_migration=parsed_args.block_migration, + disk_over_commit=parsed_args.disk_overcommit, ) else: server.migrate() |