Merge "Add --ha option to os router create command"

Author: Jenkins

doc/source/command-objects/router.rst

@@ -63,7 +63,7 @@ Create new router
     os router create
         [--project <project> [--project-domain <project-domain>]]
         [--enable | --disable]
-        [--distributed]
+        [--distributed] [--ha]
         [--description <description>]
         [--availability-zone-hint <availability-zone>]
         <name>
@@ -89,6 +89,10 @@ Create new router
 
     Create a distributed router
 
+.. option:: --ha
+
+    Create a highly available router
+
 .. option:: --description <description>
 
     Set router description

openstackclient/network/v2/router.py

@@ -94,7 +94,6 @@ def _get_attrs(client_manager, parsed_args):
         ).id
         attrs['tenant_id'] = project_id
 
-    # TODO(tangchen): Support getting 'ha' property.
     # TODO(tangchen): Support getting 'external_gateway_info' property.
 
     return attrs
@@ -180,10 +179,15 @@ def get_parser(self, prog_name):
             default=False,
             help=_("Create a distributed router")
         )
+        parser.add_argument(
+            '--ha',
+            action='store_true',
+            help=_("Create a highly available router")
+        )
         parser.add_argument(
             '--description',
             metavar='<description>',
-            help=_('Set router description')
+            help=_("Set router description")
         )
         parser.add_argument(
             '--project',
@@ -207,6 +211,8 @@ def take_action(self, parsed_args):
         client = self.app.client_manager.network
 
         attrs = _get_attrs(self.app.client_manager, parsed_args)
+        if parsed_args.ha:
+            attrs['ha'] = parsed_args.ha
         obj = client.create_router(**attrs)
 
         columns = _get_columns(obj)

openstackclient/tests/unit/network/v2/test_router.py

@@ -166,6 +166,7 @@ def test_create_default_options(self):
             ('name', self.new_router.name),
             ('enable', True),
             ('distributed', False),
+            ('ha', False),
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
@@ -178,6 +179,29 @@ def test_create_default_options(self):
         self.assertEqual(self.columns, columns)
         self.assertEqual(self.data, data)
 
+    def test_create_with_ha_option(self):
+        arglist = [
+            '--ha',
+            self.new_router.name,
+        ]
+        verifylist = [
+            ('name', self.new_router.name),
+            ('enable', True),
+            ('distributed', False),
+            ('ha', True),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = (self.cmd.take_action(parsed_args))
+
+        self.network.create_router.assert_called_once_with(**{
+            'admin_state_up': True,
+            'name': self.new_router.name,
+            'ha': True,
+        })
+        self.assertEqual(self.columns, columns)
+        self.assertEqual(self.data, data)
+
     def test_create_with_AZ_hints(self):
         arglist = [
             self.new_router.name,
@@ -189,6 +213,7 @@ def test_create_with_AZ_hints(self):
             ('availability_zone_hints', ['fake-az', 'fake-az2']),
             ('enable', True),
             ('distributed', False),
+            ('ha', False)
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 

releasenotes/notes/bug-1610161-7c34c7b735701bd4.yaml

@@ -0,0 +1,5 @@
+---
+features:
+  - |
+    Add ``--ha`` option to ``router create`` command.
+    [Bug `1610161 <https://bugs.launchpad.net/bugs/1610161>`_]