Merge "Add project field in image list subcommand"

Author: Zuul

openstackclient/image/v2/image.py

@@ -591,6 +591,12 @@ def get_parser(self, prog_name):
                     "The supported options are: %s. ") %
                   ', '.join(MEMBER_STATUS_CHOICES))
         )
+        parser.add_argument(
+            '--project',
+            metavar='<project>',
+            help=_("Search by project (admin only) (name or ID)")
+        )
+        common.add_project_domain_option_to_parser(parser)
         parser.add_argument(
             '--tag',
             metavar='<tag>',
@@ -636,6 +642,7 @@ def get_parser(self, prog_name):
         return parser
 
     def take_action(self, parsed_args):
+        identity_client = self.app.client_manager.identity
         image_client = self.app.client_manager.image
 
         kwargs = {}
@@ -659,6 +666,14 @@ def take_action(self, parsed_args):
             kwargs['member_status'] = parsed_args.member_status
         if parsed_args.tag:
             kwargs['tag'] = parsed_args.tag
+        project_id = None
+        if parsed_args.project:
+            project_id = common.find_project(
+                identity_client,
+                parsed_args.project,
+                parsed_args.project_domain,
+            ).id
+            kwargs['owner'] = project_id
         if parsed_args.long:
             columns = (
                 'ID',

openstackclient/tests/unit/image/v2/test_image.py

@@ -769,6 +769,21 @@ def test_image_list_limit_option(self):
         self.assertEqual(self.columns, columns)
         self.assertEqual(ret_limit, len(tuple(data)))
 
+    def test_image_list_project_option(self):
+        self.client.find_image = mock.Mock(return_value=self._image)
+        arglist = [
+            '--project', 'nova',
+        ]
+        verifylist = [
+            ('project', 'nova'),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = self.cmd.take_action(parsed_args)
+
+        self.assertEqual(self.columns, columns)
+        self.assertItemsEqual(self.datalist, tuple(data))
+
     @mock.patch('osc_lib.utils.find_resource')
     def test_image_list_marker_option(self, fr_mock):
         # tangchen: Since image_fakes.IMAGE is a dict, it cannot offer a .id

releasenotes/notes/add-image-project-filter-support-ed6204391e503adf.yaml

@@ -0,0 +1,5 @@
+---
+features:
+  - |
+    Add ``--project`` and ``--project-domain``options to ``image list``
+    command to filter by owner.