crccheck / cloudwatch-to-graphite

Helper for pushing AWS CloudWatch metrics to Graphite
Apache License 2.0
55 stars 28 forks source link

Should period be in minutes or seconds? #10

Open crccheck opened 10 years ago

crccheck commented 10 years ago

It's supposed to be seconds, but since they're limited to being multiples of 60, I figure minutes is human friendlier. But juggling two separate systems of numbers is annoying.

kali-hernandez commented 9 years ago

If, like me, you are planning to use leadbutt to integrate it with your collectd scripts, it becomes a bit of annoyance to have to convert in between minutes and seconds. Since aws-cli and other utilities use seconds as unit (regardless of the limitation of multiples of 60) I believe it would be nicer to express it in seconds.

An alternative would be to allow for units in the parameter (such as -p 60s)

crccheck commented 9 years ago

:+1: If there's a human friendly format, it should be REALLY human friendly and have units.