Specify bucket prefix on commandline#1
Conversation
|
the indentation got messed up by your editor or something. |
|
Fixed! |
|
i'm confused. this makes the prefix configurable, but yet it automatically adds "smokeping" to the metric name. also why smokeping, its not a ping test and the project name is smoketcp |
|
also, so this runs for you right? because i notice also the statsd object is now created differently |
|
You're correct, I should remove the smokeping part, and will do so. I modified it to use statsd.New instead of statsd.Dial, but I will change it back to minimize changes. |
|
Ah, it's not dropping it in the right place, I didn't understand how the statsd module worked, let me modify it again, sorry. |
…om all the other functions, output appears correct now
|
Ok, I believe it should now work as advertised, sorry for the back and forth. |
|
This is functional, and I have another pull request that will stack on top of it, adding flags for the command line, and allowing you to set the interval time for the tests, would you like a second pull request or for me to merge them into this one? |
This modification allows you to specify the bucket to send stats to on the command line, useful for people like me who don't like the default!