yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #21687
[Bug 1224921] Re: notification.info_from_instance should use get()
Long incomplete bug, marking as Opinion. It might be valid, but it's not
anything actionable.
** Changed in: nova
Status: Incomplete => Invalid
** Changed in: nova
Status: Invalid => Opinion
** Changed in: nova
Importance: Undecided => Wishlist
--
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/1224921
Title:
notification.info_from_instance should use get()
Status in OpenStack Compute (Nova):
Opinion
Bug description:
notification.info_from_instance() reads many values from the instance
structure, including capacity vales that are populated from the
instance_system_metadata table. However there are cases where these
values are not present - for example if a deleted instance is passed
in then the DB queries do not always do the joins. This results in a
KeyError exception.
Whilst such cases are triggered by bugs the notification code should
be more robust and use .get() methods instead.
To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1224921/+subscriptions