Merge "image: Fail if we can't find an image"

Author: Zuul

openstackclient/image/v2/image.py

@@ -191,7 +191,8 @@ def take_action(self, parsed_args):
         ).id
 
         image = image_client.find_image(
-            parsed_args.image, ignore_missing=False
+            parsed_args.image,
+            ignore_missing=False,
         )
 
         obj = image_client.add_member(
@@ -664,7 +665,8 @@ def take_action(self, parsed_args):
         for image in parsed_args.images:
             try:
                 image_obj = image_client.find_image(
-                    image, ignore_missing=False
+                    image,
+                    ignore_missing=False,
                 )
                 image_client.delete_image(image_obj.id)
             except Exception as e:
@@ -835,7 +837,10 @@ def take_action(self, parsed_args):
         if parsed_args.limit:
             kwargs['limit'] = parsed_args.limit
         if parsed_args.marker:
-            kwargs['marker'] = image_client.find_image(parsed_args.marker).id
+            kwargs['marker'] = image_client.find_image(
+                parsed_args.marker,
+                ignore_missing=False,
+            ).id
         if parsed_args.name:
             kwargs['name'] = parsed_args.name
         if parsed_args.status:
@@ -932,7 +937,10 @@ def take_action(self, parsed_args):
         image_client = self.app.client_manager.image
         columns = ("Image ID", "Member ID", "Status")
 
-        image_id = image_client.find_image(parsed_args.image).id
+        image_id = image_client.find_image(
+            parsed_args.image,
+            ignore_missing=False,
+        ).id
 
         data = image_client.members(image=image_id)
 
@@ -977,7 +985,8 @@ def take_action(self, parsed_args):
         ).id
 
         image = image_client.find_image(
-            parsed_args.image, ignore_missing=False
+            parsed_args.image,
+            ignore_missing=False,
         )
 
         image_client.remove_member(member=project_id, image=image.id)
@@ -1003,7 +1012,10 @@ def get_parser(self, prog_name):
 
     def take_action(self, parsed_args):
         image_client = self.app.client_manager.image
-        image = image_client.find_image(parsed_args.image)
+        image = image_client.find_image(
+            parsed_args.image,
+            ignore_missing=False,
+        )
 
         output_file = parsed_args.filename
         if output_file is None:
@@ -1371,7 +1383,8 @@ def take_action(self, parsed_args):
         image_client = self.app.client_manager.image
 
         image = image_client.find_image(
-            parsed_args.image, ignore_missing=False
+            parsed_args.image,
+            ignore_missing=False,
         )
 
         info = _format_image(image, parsed_args.human_readable)
@@ -1415,7 +1428,8 @@ def get_parser(self, prog_name):
     def take_action(self, parsed_args):
         image_client = self.app.client_manager.image
         image = image_client.find_image(
-            parsed_args.image, ignore_missing=False
+            parsed_args.image,
+            ignore_missing=False,
         )
 
         kwargs = {}

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

@@ -905,7 +905,10 @@ def test_image_list_marker_option(self, fr_mock):
             marker=self._image.id,
         )
 
-        self.client.find_image.assert_called_with('graven')
+        self.client.find_image.assert_called_with(
+            'graven',
+            ignore_missing=False,
+        )
 
     def test_image_list_name_option(self):
         arglist = [