covermymeds / puppet-sentry

Install and manage Sentry with Puppet!
MIT License
7 stars 7 forks source link

add metrics support for the statsd backend #32

Closed adamcstephens closed 8 years ago

adamcstephens commented 8 years ago

This allows for configuring the statsd metrics backend and the host/port to which stats are shipped.

skpy commented 8 years ago

This looks fine. Please update the README and provide parameter descriptions in the class comments as necessary.

I'd like to see all of the parameters get validators applied to them. If you're going to add validators for the new params, but not the old ones, please open an issue reminding us to bring parity to them all.