Merge "Implement a function to check the image status"

ironic/common/glance_service/base_image_service.py

@@ -183,6 +183,7 @@ class BaseImageService(object):
         :returns: A dict containing image metadata.
 
         :raises: ImageNotFound
+        :raises: ImageUnacceptable if the image status is not active
         """
         LOG.debug("Getting image metadata from glance. Image: %s",
                   image_href)
@@ -190,6 +191,11 @@ class BaseImageService(object):
 
         image = self.call(method, image_id)
 
+        if not service_utils.is_image_active(image):
+            raise exception.ImageUnacceptable(
+                image_id=image_id,
+                reason=_("The image is required to be in an active state."))
+
         if not service_utils.is_image_available(self.context, image):
             raise exception.ImageNotFound(image_id=image_id)
 

ironic/common/glance_service/service_utils.py

@@ -171,6 +171,15 @@ def is_image_available(context, image):
     return str(user_id) == str(context.user_id)
 
 
+def is_image_active(image):
+    """Check the image status.
+
+    This check is needed in case the Glance image is stuck in queued status
+    or pending_delete.
+    """
+    return str(getattr(image, 'status', None)) == "active"
+
+
 def is_glance_image(image_href):
     if not isinstance(image_href, six.string_types):
         return False

ironic/tests/unit/common/test_glance_service.py

@@ -105,7 +105,7 @@ class TestGlanceImageService(base.TestCase):
     def _make_fixture(**kwargs):
         fixture = {'name': None,
                    'properties': {},
-                   'status': None,
+                   'status': "active",
                    'is_public': None}
         fixture.update(kwargs)
         return stubs.FakeImage(fixture)
@@ -145,7 +145,7 @@ class TestGlanceImageService(base.TestCase):
             'updated_at': None,
             'deleted_at': None,
             'deleted': None,
-            'status': None,
+            'status': "active",
             'properties': {},
             'owner': None,
         }