Skip to content
This repository has been archived by the owner on Dec 11, 2023. It is now read-only.

Not able to define 'source' in Timed Aggregator #26

Closed
rdssoni opened this issue Jun 28, 2012 · 5 comments
Closed

Not able to define 'source' in Timed Aggregator #26

rdssoni opened this issue Jun 28, 2012 · 5 comments

Comments

@rdssoni
Copy link

rdssoni commented Jun 28, 2012

I am using Librato Timed Aggregator to send metrics to Librato. I use the following code :

timed_performance_aggregator = Librato::Metrics::Aggregator.new(:autosubmit_interval => 60)

But i also want to define source for the each metric sent to Librato. I am using the following command for that :

timed_performance_aggregator.add 'view_runtime' => {:source=>'My source',:value=>654.87}

The above command does not work in the sense that the metric is submitted to Librato but source is always showing 'undefined'.

Although when i define source when defining timed aggregator like :

timed_performance_aggregator = Librato::Metrics::Aggregator.new(:source=>'My source')

it works.

Please tell me if I am doing anything wrong.

@nextmat
Copy link
Contributor

nextmat commented Jun 28, 2012

@rdssoni You aren't doing anything wrong, looks like aggregator silently eats per-measurement sources right now. This is definitely a bug, we'll work on a fix for this.

@rdssoni
Copy link
Author

rdssoni commented Jul 5, 2012

Thanx @nextmat ..For looking into this..
I'll wait till it is fixed..!!

@nextmat
Copy link
Contributor

nextmat commented Jul 11, 2012

No problem, sorry for the delay. This is a slightly more complex issue than it seems like it might be, just wanted to let you know a fix is still coming.

@nextmat
Copy link
Contributor

nextmat commented Jul 25, 2012

This is resolved in 0.7.0, released yesterday. Let me know if you have any further issues?

@nextmat nextmat closed this as completed Jul 25, 2012
@rdssoni
Copy link
Author

rdssoni commented Jul 27, 2012

Thanx @nextmat..
I am currently busy in some project..i will verify this shortly and will let you know..thnx..!!

This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants