Skip to content

Commit

Permalink
DM: fix for properly idetifying VMI ownership
Browse files Browse the repository at this point in the history
If VMI is associated to a LI, then it is a BMS case.
Closes-Bug: #1506589

Change-Id: Id7f845469ff6a9fb6f0c5658b3d7431ae7e7901b
  • Loading branch information
sbalineni committed Jun 23, 2016
1 parent 7190d2d commit 025b63e
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/config/device-manager/device_manager/db.py
Expand Up @@ -1019,7 +1019,6 @@ def update(self, obj=None):
self.params = obj['virtual_machine_interface_properties']
self.service_interface_type = self.params.get(
'service_interface_type', None)
self.device_owner = obj.get("virtual_machine_interface_device_owner")
self.update_single_ref('logical_interface', obj)
self.update_single_ref('virtual_network', obj)
self.update_single_ref('floating_ip', obj)
Expand All @@ -1035,8 +1034,7 @@ def update(self, obj=None):
# end update

def is_device_owner_bms(self):
if (not self.device_owner or
self.device_owner.lower() == 'physicalrouter'):
if self.logical_interface:
return True
return False
# end
Expand Down

0 comments on commit 025b63e

Please sign in to comment.