librato / statsd-cookbook

Chef cookbook to install Statsd
Apache License 2.0
26 stars 36 forks source link

add feature to enable/disable legacy namespacing #20

Closed StephanX closed 10 years ago

StephanX commented 10 years ago

@mheffner,

So after a couple hours of tracking down "where are my lost statistics," I was pointed to this 'feature.'

https://github.com/etsy/statsd/blob/master/docs/namespacing.md

Essentially, namespacing conventions differ between pre 0.50 and 0.50+. This PR enables a toggle to choose which convention is preferable.

-S

StephanX commented 10 years ago

Ah, right on. I've pushed an update that will only set the key if graphite is enabled.

StephanX commented 10 years ago

Didn't in testing (which is why I didn't notice it) but have removed. Good catch!

mheffner commented 10 years ago

Thanks!