set default source to something meaningful

Bug #1070857 reported by Doug Hellmann
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Low
Julien Danjou

Bug Description

All of the counters use "?" as the default source. We should make that something meaningful, like "OpenStack" or "ceilometer".

Revision history for this message
Nick Barcet (nijaba) wrote :

IMHO, something usefull would actually be the API enpoint for the keystone that is used to manage identities.

Revision history for this message
Julien Danjou (jdanjou) wrote :

Not sure that's enough if you have multiple region for example, that won't help identifying which region the instance id for example come from.

Just thinking out loud. :)

Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

Can we get something like that out of the notification messages? We'll have an endpoint for polling, but it might be different. Would that matter?

Revision history for this message
Julien Danjou (jdanjou) wrote :

All we have in notifications that looks like that would be "publisher_id", but it's really not that useful and interesting as a source.

I think more at source like the platform your getting data from. Maybe that should simply be a configuration option that is used by the collector and agent to put in their meter messages?
So if you run an agent on 2 different platforms for the same openstack, each one can be configured to be a different source.

Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

That may be the best solution. If we do go that route, we should remove the source field from Counter and make it an argument to the publish function so all of the plugins aren't tied to that config setting.

Revision history for this message
Julien Danjou (jdanjou) wrote :

I agree. I volunteer to make the change if nobody objects.

Revision history for this message
Nick Barcet (nijaba) wrote :

+1 but would recommend that you bring up the question in the meeting before jumping to the solution, as others may have great ideas :)

Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

+1 on getting other feedback and having Julien do the work.

Revision history for this message
Julien Danjou (jdanjou) wrote :

Ack!

Changed in ceilometer:
assignee: nobody → Julien Danjou (jdanjou)
Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

See also #1038043, which I closed as a dupe of this one.

Julien Danjou (jdanjou)
Changed in ceilometer:
status: Triaged → Fix Committed
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: none → grizzly-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: grizzly-2 → 2013.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.