Merge "Add parent project filter for listing projects"

Zuul · openstack-gerrit · commit 7e98aaefa9f8 · 2019-10-22T21:16:40.000Z
diff --git a/doc/source/cli/command-objects/project.rst b/doc/source/cli/command-objects/project.rst
@@ -102,6 +102,7 @@ List projects
 
     openstack project list
         [--domain <domain>]
+        [--parent <parent>]
         [--user <user>]
         [--my-projects]
         [--long]
@@ -115,6 +116,12 @@ List projects
 
     .. versionadded:: 3
 
+.. option:: --parent <parent>
+
+    Filter projects whose parent is :option:`\<parent\> <--parent>` (name or ID)
+
+    .. versionadded:: 3
+
 .. option:: --user <user>
 
     Filter projects by :option:`\<user\> <--user>` (name or ID)
diff --git a/openstackclient/identity/v3/project.py b/openstackclient/identity/v3/project.py
@@ -185,6 +185,11 @@ def get_parser(self, prog_name):
             metavar='<domain>',
             help=_('Filter projects by <domain> (name or ID)'),
         )
+        parser.add_argument(
+            '--parent',
+            metavar='<parent>',
+            help=_('Filter projects whose parent is <parent> (name or ID)'),
+        )
         parser.add_argument(
             '--user',
             metavar='<user>',
@@ -226,6 +231,11 @@ def take_action(self, parsed_args):
                                            parsed_args.domain).id
             kwargs['domain'] = domain_id
 
+        if parsed_args.parent:
+            parent_id = common.find_project(identity_client,
+                                            parsed_args.parent).id
+            kwargs['parent'] = parent_id
+
         if parsed_args.user:
             if parsed_args.domain:
                 user_id = utils.find_resource(identity_client.users,
diff --git a/openstackclient/tests/unit/identity/v3/test_project.py b/openstackclient/tests/unit/identity/v3/test_project.py
@@ -645,6 +645,28 @@ def test_project_list_domain_no_perms(self):
         self.assertEqual(self.columns, columns)
         self.assertEqual(self.datalist, tuple(data))
 
+    def test_project_list_parent(self):
+        self.parent = identity_fakes.FakeProject.create_one_project()
+        self.project = identity_fakes.FakeProject.create_one_project(
+            attrs={'domain_id': self.domain.id, 'parent_id': self.parent.id})
+
+        arglist = [
+            '--parent', self.parent.id,
+        ]
+        verifylist = [
+            ('parent', self.parent.id),
+        ]
+
+        self.projects_mock.get.return_value = self.parent
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = self.cmd.take_action(parsed_args)
+        self.projects_mock.list.assert_called_with(parent=self.parent.id)
+
+        self.assertEqual(self.columns, columns)
+        self.assertEqual(self.datalist, tuple(data))
+
     def test_project_list_sort(self):
         self.projects_mock.list.return_value = self.projects
 
diff --git a/releasenotes/notes/add-parent-list-option-to-projects-10382a7176993366.yaml b/releasenotes/notes/add-parent-list-option-to-projects-10382a7176993366.yaml
@@ -0,0 +1,5 @@
+---
+features:
+  - |
+    Add ``--parent`` option to ``project list`` command to filter projects
+    by the specified parent project.
