diff options
| -rw-r--r-- | .mailmap | 1 | ||||
| -rw-r--r-- | Authors | 2 | ||||
| -rw-r--r-- | nova/db/sqlalchemy/api.py | 2 | ||||
| -rw-r--r-- | nova/tests/test_db_api.py | 2 |
4 files changed, 5 insertions, 2 deletions
@@ -79,3 +79,4 @@ <nsokolov@griddynamics.com> <nsokolov@griddynamics.net> <troy.toman@rackspace.com> <ttcl@mac.com> <zulcss@ubuntu.com> <chuck.short@canonical.com> +<jhtran@att.com> <jtran@attinteractive.com> @@ -99,7 +99,7 @@ John Dewey <john@dewey.ws> John Garbutt <john.garbutt@citrix.com> John Griffith <john.griffith@solidfire.com> John Kennedy <john.m.kennedy@intel.com> -John Tran <jtran@attinteractive.com> +John Tran <jhtran@att.com> Jonathan Bryce <jbryce@jbryce.com> Jordan Rinke <jordan@openstack.org> Joseph Suh <jsuh@isi.edu> diff --git a/nova/db/sqlalchemy/api.py b/nova/db/sqlalchemy/api.py index 3a4e8ea1b..c555c73c8 100644 --- a/nova/db/sqlalchemy/api.py +++ b/nova/db/sqlalchemy/api.py @@ -1070,7 +1070,7 @@ def fixed_ip_get_all(context, session=None): @require_context def fixed_ip_get_by_address(context, address, session=None): result = model_query(context, models.FixedIp, session=session, - read_deleted="yes").\ + read_deleted=context.read_deleted).\ filter_by(address=address).\ first() if not result: diff --git a/nova/tests/test_db_api.py b/nova/tests/test_db_api.py index c1002d98a..1abd44a90 100644 --- a/nova/tests/test_db_api.py +++ b/nova/tests/test_db_api.py @@ -151,6 +151,8 @@ class DbApiTestCase(test.TestCase): db.network_delete_safe, ctxt, network['id']) db.fixed_ip_update(ctxt, address2, {'allocated': False}) network = db.network_delete_safe(ctxt, network['id']) + self.assertRaises(exception.FixedIpNotFoundForAddress, + db.fixed_ip_get_by_address, ctxt, address1) ctxt = ctxt.elevated(read_deleted='yes') fixed_ip = db.fixed_ip_get_by_address(ctxt, address1) self.assertTrue(fixed_ip['deleted']) |