[openstack-dev] [nova][ceilometer] ceilometer unit tests broke because of a nova patch
Dan Smith
dms at danplanet.com
Wed Feb 5 14:57:14 UTC 2014
> We don't have to add a new notification, but we have to add some
> new datas in the nova notifications. At least for the delete
> instance notification to remove the ceilometer nova notifier.
>
> A while ago, I have registered a blueprint that explains which
> datas are missing in the current nova notifications:
>
> https://blueprints.launchpad.net/nova/+spec/usage-data-in-notification
>
>
https://wiki.openstack.org/wiki/Ceilometer/blueprints/remove-ceilometer-nova-notifier
This seems like a much better way to do this.
I'm not opposed to a nova plugin, but if it's something that lives
outside the nova tree, I think there's going to be a problem of
constantly chasing internal API changes. IMHO, a plugin should live
(and be tested) in the nova tree and provide/consume a stableish API
to/from Ceilometer.
So, it seems like we've got the following options:
1. Provide the required additional data in our notifications to avoid
the need for a plugin to hook into nova internals.
2. Continue to use a plugin in nova to scrape the additional data
needed during certain events, but hopefully in a way that ties the
plugin to the internal APIs in a maintainable way.
Is that right?
Personally, I think #1 is far superior to #2.
--Dan
More information about the OpenStack-dev
mailing list