← Back to team overview

openstack team mailing list archive

Re: Notifiers

 

On 03/12/2012 11:56 AM, Johannes Erdfelt wrote:
On Mon, Mar 12, 2012, Jay Pipes<jaypipes@xxxxxxxxx>  wrote:
On 03/12/2012 09:33 AM, Swaminathan Venkataraman wrote:
I've been playing around with openstack for a month now and was looking
to see how I can contribute. I saw that nova and glance use different
methods to send out notifications. It looked relatively straight forward
to make them use one common library for notifications. That way, when we
make changes or add new transports we do not have to do it twice (or
more number of times depending on how other components of openstack do
notifications). Let me know if someone is already working on this. If
not, let me know if this makes sense and I can start by creating a
blueprint. Also, is there any preference to use object oriented (or not)?

It is a surprise that the code is different for event notifications
between Nova and Glance. I believe the same developer (Johannes
Erdfelt) did them both. And I think his plan was to put the code
into the openstack-common library.

Johannes, thoughts?

I only did the work for Glance and it was based on the Nova code at the
time. I guess the code has diverged in the 7 months since I did that
work.

Ah, my mistake, thanks for the heads up.

I'm fine with getting it into common code, but I have no plans to do it
myself.

OK.

Yes, I too would be perfectly happy to see the code moved to openstack-common.

Anybody interested in taking on that task?

-jay


Follow ups

References