← Back to team overview

yahoo-eng-team team mailing list archive

[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