x/python-ganttclient
Code Issues Proposed changes

migration_get_all_unconfirmed() now uses lowercase "finished" Fixes bug

Browse Source 
977719

status set in _finish_resize is lower case, but
migration_get_all_unconfirmed() was filtering using FINISHED
Also updated corresponding test case to use lower case "finished"
Updated Authors file

Change-Id: Ia30e66e541bba91a9dd3d3af1429b2312870793a
This commit is contained in:
Mandar Vaze
2012-04-10 05:05:17 -07:00
parent 570db48243
commit ec51239fbf
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
Authors
View File

@@ -124,6 +124,7 @@ Loganathan Parthipan <parthipan@hp.com>
Lorin Hochstein <lorin@nimbisservices.com> Lorin Hochstein <lorin@nimbisservices.com>
Lvov Maxim <usrleon@gmail.com> Lvov Maxim <usrleon@gmail.com>
Mandell Degerness <mdegerne@gmail.com> Mandell Degerness <mdegerne@gmail.com>
Mandar Vaze <mandar.vaze@vertex.co.in>
Mark McClain <mark.mcclain@dreamhost.com> Mark McClain <mark.mcclain@dreamhost.com>
Mark McLoughlin <markmc@redhat.com> Mark McLoughlin <markmc@redhat.com>
Mark Washenberger <mark.washenberger@rackspace.com> Mark Washenberger <mark.washenberger@rackspace.com>

4
nova/tests/test_db_api.py
View File

@@ -89,7 +89,7 @@ class DbApiTestCase(test.TestCase):
# Ensure one migration older than 10 seconds is returned. # Ensure one migration older than 10 seconds is returned.
updated_at = datetime.datetime(2000, 01, 01, 12, 00, 00) updated_at = datetime.datetime(2000, 01, 01, 12, 00, 00)
values = {"status": "FINISHED", "updated_at": updated_at} values = {"status": "finished", "updated_at": updated_at}
migration = db.migration_create(ctxt, values) migration = db.migration_create(ctxt, values)
results = db.migration_get_all_unconfirmed(ctxt, 10) results = db.migration_get_all_unconfirmed(ctxt, 10)
self.assertEqual(1, len(results)) self.assertEqual(1, len(results))
@@ -97,7 +97,7 @@ class DbApiTestCase(test.TestCase):
# Ensure the new migration is not returned. # Ensure the new migration is not returned.
updated_at = datetime.datetime.utcnow() updated_at = datetime.datetime.utcnow()
values = {"status": "FINISHED", "updated_at": updated_at} values = {"status": "finished", "updated_at": updated_at}
migration = db.migration_create(ctxt, values) migration = db.migration_create(ctxt, values)
results = db.migration_get_all_unconfirmed(ctxt, 10) results = db.migration_get_all_unconfirmed(ctxt, 10)
self.assertEqual(0, len(results)) self.assertEqual(0, len(results))