<br><br><div class="gmail_quote">On Sun, Nov 18, 2012 at 5:07 AM, Jiang, Yunhong <span dir="ltr"><<a href="mailto:yunhong.jiang@intel.com" target="_blank">yunhong.jiang@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi, Doug<br>
        As a follow-up for the discussion on nova_notifier yesterday. I checked nova code, seems it support extra information as followed declaration.<br>
    def _notify_about_instance_usage(self, context, instance, event_suffix,<br>
                                     network_info=None, system_metadata=None,<br>
                                     extra_usage_info=None):<br>
<br>
So after we have the option #5 library in nova, we can pass the metering result as extra information to the listener in compute.instance.delete.start event. With this changes, I think we don't need the nova_notifier anymore.<br>
</blockquote><div><br></div><div>We would need to make the change everywhere instance-related events are generated in nova so that the metadata for those events is always the same. I think it would be better to change info_from_instance() in nova/notifications.py so that the callers don't have to be updated, but I don't know if that module would have the information it needs to collect the appropriate stats from the hypervisor. For events generated when the instance doesn't exist we would want to substitute suitable constants (probably zero).</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
        Background: currently ceilometer provide a nova notification plugin to notify ceilometer when instance being deleted, see more information at <a href="https://bugs.launchpad.net/ceilometer/+bug/1005944" target="_blank">https://bugs.launchpad.net/ceilometer/+bug/1005944</a>. Discussed with Doug and possibly we can remove it if nova provides the metering information in the compute.instance.delete.start event notification.<br>
</blockquote><div><br></div><div>The notification plugin generates meter messages about usage right before the instance is deleted to avoid a race condition between the time the delete notification is sent and when the ceilometer agent can poll for those values itself. We process the regular notification as a separate event.</div>
<div><br></div><div>Doug</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks<br>
--jyh<br>
</blockquote></div><br>