diff options
| author | venkata anil <anilvenkata@redhat.com> | 2017-02-14 05:05:20 +0000 |
|---|---|---|
| committer | Dean Troyer <dtroyer@gmail.com> | 2017-04-06 14:37:14 +0000 |
| commit | fe59e339ae6dd6a5e9075773fb5c2a0fea9c2e53 (patch) | |
| tree | 15a357a4cc389bb0e73bc2c0f2df9f4a39002f3a /openstackclient/network | |
| parent | 27f96a3a7a5515ff621300a917872b5ec62f41de (diff) | |
| download | python-openstackclient-fe59e339ae6dd6a5e9075773fb5c2a0fea9c2e53.tar.gz | |
Allow override of distributed router flag
When router_distributed=True is set in neutron.conf,
user can't override this and create a centralized router
through openstack client. Openstack client allows modifying
routers from distributed to centralized after creation but
not supporting centralized flag during creation. We allow
centralized and distributed flags during router creation
with this change.
Closes-bug: #1664255
Change-Id: I75f72ca695338ad8c381cfa89fbb9d8e61ee7bc5
Diffstat (limited to 'openstackclient/network')
| -rw-r--r-- | openstackclient/network/v2/router.py | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/openstackclient/network/v2/router.py b/openstackclient/network/v2/router.py index f322d5d1..0da91baa 100644 --- a/openstackclient/network/v2/router.py +++ b/openstackclient/network/v2/router.py @@ -78,8 +78,7 @@ def _get_attrs(client_manager, parsed_args): attrs['admin_state_up'] = True if parsed_args.disable: attrs['admin_state_up'] = False - # centralized is available only for SetRouter and not for CreateRouter - if 'centralized' in parsed_args and parsed_args.centralized: + if parsed_args.centralized: attrs['distributed'] = False if parsed_args.distributed: attrs['distributed'] = True @@ -176,13 +175,17 @@ class CreateRouter(command.ShowOne): action='store_true', help=_("Disable router") ) - parser.add_argument( + distribute_group = parser.add_mutually_exclusive_group() + distribute_group.add_argument( '--distributed', - dest='distributed', action='store_true', - default=False, help=_("Create a distributed router") ) + distribute_group.add_argument( + '--centralized', + action='store_true', + help=_("Create a centralized router") + ) ha_group = parser.add_mutually_exclusive_group() ha_group.add_argument( '--ha', |