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

Author: voyageur

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:

openstackclient/network/v2/network_agent.py

@@ -351,7 +351,7 @@ def take_action(self, parsed_args):
         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

openstackclient/network/v2/network_qos_policy.py

@@ -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:

openstackclient/network/v2/port.py

@@ -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:

openstackclient/network/v2/router.py

@@ -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:

openstackclient/network/v2/subnet.py

@@ -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:

openstackclient/network/v2/subnet_pool.py

@@ -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: