diff options
| author | Jenkins <jenkins@review.openstack.org> | 2012-03-07 00:19:19 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2012-03-07 00:19:19 +0000 |
| commit | d524ea2ea5c9c81d41e17f6b2359f0b0edc3f800 (patch) | |
| tree | 59ba8a1584c08ceeb315eeb0180c2e7262e5488e /nova/db | |
| parent | 0bd3b4703240a41c09fe6926348cd4d10df4acae (diff) | |
| parent | 84114e426f3973d78fc0f6766f3ad7577349da7a (diff) | |
Merge "Update fixed_ip_associate to not use relationships"
Diffstat (limited to 'nova/db')
| -rw-r--r-- | nova/db/sqlalchemy/api.py | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/nova/db/sqlalchemy/api.py b/nova/db/sqlalchemy/api.py index 635071b29..07d48a229 100644 --- a/nova/db/sqlalchemy/api.py +++ b/nova/db/sqlalchemy/api.py @@ -993,16 +993,12 @@ def fixed_ip_associate(context, address, instance_id, network_id=None, if fixed_ip_ref is None: raise exception.FixedIpNotFoundForNetwork(address=address, network_id=network_id) - if fixed_ip_ref.instance is not None: + if fixed_ip_ref.instance_id: raise exception.FixedIpAlreadyInUse(address=address) - if not fixed_ip_ref.network: - fixed_ip_ref.network = network_get(context, - network_id, - session=session) - fixed_ip_ref.instance = instance_get(context, - instance_id, - session=session) + if not fixed_ip_ref.network_id: + fixed_ip_ref.network_id = network_id + fixed_ip_ref.instance_id = instance_id session.add(fixed_ip_ref) return fixed_ip_ref['address'] |
