diff options
| author | Bernard Cafarelli <bcafarel@redhat.com> | 2019-01-28 14:17:46 +0100 |
|---|---|---|
| committer | Bernard Cafarelli <bcafarel@redhat.com> | 2019-02-13 10:43:11 +0100 |
| commit | 05521bf84cb108c73bb36b270569b1986ad13f53 (patch) | |
| tree | b71e048988591919e0d79c61fcbc155acfc19861 /openstackclient/network/v2/network.py | |
| parent | 1a0bef2b46c76e1c6308def709199c30c68aae47 (diff) | |
| download | python-openstackclient-05521bf84cb108c73bb36b270569b1986ad13f53.tar.gz | |
Remove str() when setting network objects names
Most network commands use str() on name argument, which fails on python
2 with Unicode characters. This comes from parsed arguments so does not
actually need this call.
Sample command failing with current code:
openstack network create test_unicode™
Change-Id: Ie10b67864c912ee5c33e90b10c3d9705ee8307e7
Story: 2004356
Task: 27955
Diffstat (limited to 'openstackclient/network/v2/network.py')
| -rw-r--r-- | openstackclient/network/v2/network.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/openstackclient/network/v2/network.py b/openstackclient/network/v2/network.py index 0fdf62c9..f5123932 100644 --- a/openstackclient/network/v2/network.py +++ b/openstackclient/network/v2/network.py @@ -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: |