Ensure minRam and minDisk are always integers

Fixing bug 859149

Change-Id: I52bcb442225e8e3cb4aecc6d993e1c3a774cdf84

nova/api/openstack/views/images.py

@@ -197,11 +197,20 @@ class ViewBuilderV11(ViewBuilder):
 
         if detail:
             image["metadata"] = image_obj.get("properties", {})
-            if 'min_ram' in image_obj:
-                image["minRam"] = image_obj.get("min_ram") or 0
 
-            if 'min_disk' in image_obj:
-                image["minDisk"] = image_obj.get("min_disk") or 0
+            min_ram = image_obj.get('min_ram') or 0
+            try:
+                min_ram = int(min_ram)
+            except ValueError:
+                min_ram = 0
+            image['minRam'] = min_ram
+
+            min_disk = image_obj.get('min_disk') or 0
+            try:
+                min_disk = int(min_disk)
+            except ValueError:
+                min_disk = 0
+            image['minDisk'] = min_disk
 
         return image
 

nova/tests/api/openstack/fakes.py

@@ -219,7 +219,8 @@ def _make_image_fixtures():
 
     # Public image
     add_fixture(id=image_id, name='public image', is_public=True,
-                status='active', properties={'key1': 'value1'})
+                status='active', properties={'key1': 'value1'},
+                min_ram="128", min_disk="10")
     image_id += 1
 
     # Snapshot for User 1

nova/tests/api/openstack/test_images.py

@@ -659,8 +659,8 @@ class ImagesTest(test.TestCase):
             'created': NOW_API_FORMAT,
             'status': 'ACTIVE',
             'progress': 100,
-            'minDisk': 0,
-            'minRam': 0,
+            'minDisk': 10,
+            'minRam': 128,
             "links": [{
                 "rel": "self",
                 "href": "http://localhost/v1.1/fake/images/123",
@@ -950,9 +950,9 @@ class ImagesTest(test.TestCase):
             'updated': NOW_API_FORMAT,
             'created': NOW_API_FORMAT,
             'status': 'ACTIVE',
-            'minDisk': 0,
+            'minDisk': 10,
             'progress': 100,
-            'minRam': 0,
+            'minRam': 128,
             "links": [{
                 "rel": "self",
                 "href": "http://localhost/v1.1/fake/images/123",
@@ -1371,6 +1371,8 @@ class ImageXMLSerializationTest(test.TestCase):
                 'updated': self.TIMESTAMP,
                 'status': 'ACTIVE',
                 'progress': 80,
+                'minRam': 10,
+                'minDisk': 100,
                 'server': {
                     'id': '1',
                     'links': [