crccheck / cloudwatch-to-graphite

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

Add ElastiCache support for plumbum #17

Closed choopooly closed 9 years ago

crccheck commented 9 years ago

sorry I haven't had time to look at this yet, but at a glance, it looks excellent. I wish there were trailing commas though in those two dicts.

choopooly commented 9 years ago

Good, it's a pretty "beta" feature as AWS do not at the time support tags for elasticache, it can't be filtered yet. Also it doesn't support nodes id (currently hardcoded in my template) but we'll get there !

I'm not sure to understand what you meant about the trailing commas.