Merge "Remove str() when setting network objects names"
This commit is contained in:
commit
462a087bfb
openstackclient/network/v2
@ -72,7 +72,7 @@ def _get_columns_compute(item):
|
||||
def _get_attrs_network(client_manager, parsed_args):
|
||||
attrs = {}
|
||||
if parsed_args.name is not None:
|
||||
attrs['name'] = str(parsed_args.name)
|
||||
attrs['name'] = parsed_args.name
|
||||
if parsed_args.enable:
|
||||
attrs['admin_state_up'] = True
|
||||
if parsed_args.disable:
|
||||
@ -143,7 +143,7 @@ def _get_attrs_network(client_manager, parsed_args):
|
||||
def _get_attrs_compute(client_manager, parsed_args):
|
||||
attrs = {}
|
||||
if parsed_args.name is not None:
|
||||
attrs['name'] = str(parsed_args.name)
|
||||
attrs['name'] = parsed_args.name
|
||||
if parsed_args.share:
|
||||
attrs['share_subnet'] = True
|
||||
if parsed_args.no_share:
|
||||
|
@ -351,7 +351,7 @@ class SetNetworkAgent(command.Command):
|
||||
obj = client.get_agent(parsed_args.network_agent)
|
||||
attrs = {}
|
||||
if parsed_args.description is not None:
|
||||
attrs['description'] = str(parsed_args.description)
|
||||
attrs['description'] = parsed_args.description
|
||||
if parsed_args.enable:
|
||||
attrs['is_admin_state_up'] = True
|
||||
attrs['admin_state_up'] = True
|
||||
|
@ -38,7 +38,7 @@ def _get_columns(item):
|
||||
def _get_attrs(client_manager, parsed_args):
|
||||
attrs = {}
|
||||
if 'name' in parsed_args and parsed_args.name is not None:
|
||||
attrs['name'] = str(parsed_args.name)
|
||||
attrs['name'] = parsed_args.name
|
||||
if 'description' in parsed_args and parsed_args.description is not None:
|
||||
attrs['description'] = parsed_args.description
|
||||
if parsed_args.share:
|
||||
|
@ -134,7 +134,7 @@ def _get_attrs(client_manager, parsed_args):
|
||||
attrs['dns_name'] = parsed_args.dns_name
|
||||
# It is possible that name is not updated during 'port set'
|
||||
if parsed_args.name is not None:
|
||||
attrs['name'] = str(parsed_args.name)
|
||||
attrs['name'] = parsed_args.name
|
||||
# The remaining options do not support 'port set' command, so they require
|
||||
# additional check
|
||||
if 'network' in parsed_args and parsed_args.network is not None:
|
||||
|
@ -85,7 +85,7 @@ def _get_columns(item):
|
||||
def _get_attrs(client_manager, parsed_args):
|
||||
attrs = {}
|
||||
if parsed_args.name is not None:
|
||||
attrs['name'] = str(parsed_args.name)
|
||||
attrs['name'] = parsed_args.name
|
||||
if parsed_args.enable:
|
||||
attrs['admin_state_up'] = True
|
||||
if parsed_args.disable:
|
||||
|
@ -169,7 +169,7 @@ def _get_attrs(client_manager, parsed_args, is_create=True):
|
||||
attrs = {}
|
||||
client = client_manager.network
|
||||
if 'name' in parsed_args and parsed_args.name is not None:
|
||||
attrs['name'] = str(parsed_args.name)
|
||||
attrs['name'] = parsed_args.name
|
||||
|
||||
if is_create:
|
||||
if 'project' in parsed_args and parsed_args.project is not None:
|
||||
|
@ -52,7 +52,7 @@ def _get_attrs(client_manager, parsed_args):
|
||||
network_client = client_manager.network
|
||||
|
||||
if parsed_args.name is not None:
|
||||
attrs['name'] = str(parsed_args.name)
|
||||
attrs['name'] = parsed_args.name
|
||||
if parsed_args.prefixes is not None:
|
||||
attrs['prefixes'] = parsed_args.prefixes
|
||||
if parsed_args.default_prefix_length is not None:
|
||||
|
Loading…
x
Reference in New Issue
Block a user