Yelp / sensu_handlers

Custom Sensu Handlers to support a multi-tenant environment, allowing checks themselves to emit the type of handler behavior they need in the event json
Apache License 2.0
75 stars 31 forks source link

Add support for tags as labels #57

Closed jaxxstorm closed 9 years ago

jaxxstorm commented 9 years ago

Refactored as per @keymone's suggestions.

This will use the arbitrary tags introduced in https://github.com/Yelp/puppet-monitoring_check/commit/f63c963ef4d46d7c6805224d6a75f053dd5cc2bb

We might want to merge this into a another branch and test it before merging this.

solarkennedy commented 9 years ago

How about a quick test case?

mks-m commented 9 years ago

test case++

if you get any troubles with splat operator - pipe-merging array is also a good option: https://gist.github.com/keymone/5eebf33962282a7b9b0e

jaxxstorm commented 9 years ago

Yep I'll try knock some tests together.

jaxxstorm commented 9 years ago

rspec added. Let me know what you think @solarkennedy

solarkennedy commented 9 years ago

ship