Public bug reported:
def get_instance_pci_devs(inst):
"""Get the devices assigned to the instances."""
if isinstance(inst, objects.Instance):
return inst.pci_devices
else:
ctxt = context.get_admin_context()
return objects.PciDeviceList.get_by_instance_uuid(
ctxt, inst['uuid'])
In the above code, the else part may not be used by the normal code
flow. Removing it may break some of the unit tests. Thus fix is also
needed in the unit test code that is using it.
** Affects: nova
Importance: Undecided
Status: New
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1361797
Title:
unused code in pci_manager.get_instance_pci_devs()
Status in OpenStack Compute (Nova):
New
Bug description:
def get_instance_pci_devs(inst):
"""Get the devices assigned to the instances."""
if isinstance(inst, objects.Instance):
return inst.pci_devices
else:
ctxt = context.get_admin_context()
return objects.PciDeviceList.get_by_instance_uuid(
ctxt, inst['uuid'])
In the above code, the else part may not be used by the normal code
flow. Removing it may break some of the unit tests. Thus fix is also
needed in the unit test code that is using it.
To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1361797/+subscriptions
--
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help : https://help.launchpad.net/ListHelp
