Merge "Make instance object tolerate isotime strings"

nova/objects/base.py

@@ -168,9 +168,9 @@ class NovaObject(object):
     # by subclasses, but that is a special case. Objects inheriting from
     # other objects will not receive this merging of fields contents.
     fields = {
-        'created_at': obj_utils.datetime_or_none,
-        'updated_at': obj_utils.datetime_or_none,
-        'deleted_at': obj_utils.datetime_or_none,
+        'created_at': obj_utils.datetime_or_str_or_none,
+        'updated_at': obj_utils.datetime_or_str_or_none,
+        'deleted_at': obj_utils.datetime_or_str_or_none,
         }
 
     def __init__(self):

nova/objects/instance.py

@@ -58,9 +58,9 @@ class Instance(base.NovaObject):
 
         'reservation_id': obj_utils.str_or_none,
 
-        'scheduled_at': obj_utils.datetime_or_none,
-        'launched_at': obj_utils.datetime_or_none,
-        'terminated_at': obj_utils.datetime_or_none,
+        'scheduled_at': obj_utils.datetime_or_str_or_none,
+        'launched_at': obj_utils.datetime_or_str_or_none,
+        'terminated_at': obj_utils.datetime_or_str_or_none,
 
         'availability_zone': obj_utils.str_or_none,
 

nova/objects/utils.py

@@ -27,6 +27,14 @@ def datetime_or_none(dt):
     raise ValueError('A datetime.datetime is required here')
 
 
+# NOTE(danms): Being tolerant of isotime strings here will help us
+# during our objects transition
+def datetime_or_str_or_none(val):
+    if isinstance(val, basestring):
+        return timeutils.parse_isotime(val)
+    return datetime_or_none(val)
+
+
 def int_or_none(val):
     """Attempt to parse an integer value, or None."""
     if val is None:

nova/tests/objects/test_objects.py

@@ -123,6 +123,14 @@ class TestUtils(test.TestCase):
         self.assertEqual(utils.datetime_or_none(None), None)
         self.assertRaises(ValueError, utils.datetime_or_none, 'foo')
 
+    def test_datetime_or_str_or_none(self):
+        dts = timeutils.isotime()
+        dt = timeutils.parse_isotime(dts)
+        self.assertEqual(utils.datetime_or_str_or_none(dt), dt)
+        self.assertEqual(utils.datetime_or_str_or_none(None), None)
+        self.assertEqual(utils.datetime_or_str_or_none(dts), dt)
+        self.assertRaises(ValueError, utils.datetime_or_str_or_none, 'foo')
+
     def test_int_or_none(self):
         self.assertEqual(utils.int_or_none(1), 1)
         self.assertEqual(utils.int_or_none('1'), 1)