Merge "Fix i18n support for help and error msg in network"

Author: Jenkins

openstackclient/network/v2/address_scope.py

@@ -185,7 +185,7 @@ def take_action(self, parsed_args):
         if parsed_args.no_share:
             attrs['shared'] = False
         if attrs == {}:
-            msg = "Nothing specified to be set."
+            msg = _("Nothing specified to be set.")
             raise exceptions.CommandError(msg)
         client.update_address_scope(obj, **attrs)
 

openstackclient/network/v2/network.py

@@ -219,7 +219,7 @@ def update_parser_network(self, parser):
         default_router_grp.add_argument(
             '--no-default',
             action='store_true',
-            help=_("Do not use the network as the default external network. "
+            help=_("Do not use the network as the default external network "
                    "(default)")
         )
         _add_additional_network_options(parser)
@@ -256,7 +256,7 @@ def update_parser_common(self, parser):
             'network',
             metavar="<network>",
             nargs="+",
-            help=("Network(s) to delete (name or ID)")
+            help=_("Network(s) to delete (name or ID)")
         )
         return parser
 
@@ -282,13 +282,13 @@ def update_parser_common(self, parser):
             '--external',
             action='store_true',
             default=False,
-            help='List external networks',
+            help=_("List external networks")
         )
         parser.add_argument(
             '--long',
             action='store_true',
             default=False,
-            help='List additional fields in output',
+            help=_("List additional fields in output")
         )
         return parser
 
@@ -435,7 +435,7 @@ def take_action(self, parsed_args):
 
         attrs = _get_attrs(self.app.client_manager, parsed_args)
         if attrs == {}:
-            msg = "Nothing specified to be set"
+            msg = _("Nothing specified to be set")
             raise exceptions.CommandError(msg)
 
         client.update_network(obj, **attrs)

openstackclient/network/v2/port.py

@@ -414,7 +414,7 @@ def take_action(self, parsed_args):
             attrs['fixed_ips'] = []
 
         if attrs == {}:
-            msg = "Nothing specified to be set"
+            msg = _("Nothing specified to be set")
             raise exceptions.CommandError(msg)
         client.update_port(obj, **attrs)
 

openstackclient/network/v2/router.py

@@ -427,7 +427,7 @@ def take_action(self, parsed_args):
             attrs['routes'] = obj.routes + parsed_args.routes
 
         if attrs == {}:
-            msg = "Nothing specified to be set"
+            msg = _("Nothing specified to be set")
             raise exceptions.CommandError(msg)
 
         client.update_router(obj, **attrs)

openstackclient/network/v2/security_group_rule.py

@@ -519,8 +519,8 @@ def take_action_compute(self, client, parsed_args):
                 break
 
         if obj is None:
-            msg = "Could not find security group rule " \
-                  "with ID %s" % parsed_args.rule
+            msg = _("Could not find security group rule with ID ") + \
+                parsed_args.rule
             raise exceptions.CommandError(msg)
 
         # NOTE(rtheis): Format security group rule

openstackclient/network/v2/subnet.py

@@ -141,9 +141,9 @@ def _get_attrs(client_manager, parsed_args, is_create=True):
         gateway = parsed_args.gateway.lower()
 
         if not is_create and gateway == 'auto':
-            raise exceptions.CommandError("Auto option is not available"
-                                          " for Subnet Set. Valid options are"
-                                          " <ip-address> or none")
+            msg = _("Auto option is not available for Subnet Set. "
+                    "Valid options are <ip-address> or none")
+            raise exceptions.CommandError(msg)
         elif gateway != 'auto':
             if gateway == 'none':
                 attrs['gateway_ip'] = None

openstackclient/network/v2/subnet_pool.py

@@ -287,7 +287,7 @@ def take_action(self, parsed_args):
 
         attrs = _get_attrs(self.app.client_manager, parsed_args)
         if attrs == {}:
-            msg = "Nothing specified to be set"
+            msg = _("Nothing specified to be set")
             raise exceptions.CommandError(msg)
 
         # Existing prefixes must be a subset of the new prefixes.