Add support for enable/disable some checks about consul-alerts HOT 10 CLOSED

+1 on this. One of my concerns with moving all of my monitoring/alerting from Sensu to Consul is not easily being able to enable/disable alerting on checks.

I think both of lyrixx's options are valid scenarios. Either I have a node which I want to disable alerting on temporarily, or, I have a check script which is behaving undesirably on all nodes and I want to disable that check completely.

from consul-alerts.

Although it would add complexity, it might actually be useful to allow both a full black list and individual hosts (or even tags).

from consul-alerts.

I like both idea. We can possibly add both. The blacklist would have more weight than the individual config.

Basically, if it's blacklisted, ignore it right away. If not, check if it's enabled/disabled. Tags would probably be the most flexible (and more complex) solution.

from consul-alerts.

Hi guys, just release v0.1.2 with a blacklist feature. Config guide is in the README. Check it out. I think we've covered most use-cases with this. We can implement a tag based solution sometime soon.

from consul-alerts.

Huhu, I just noticed it in my logs

time="2014-11-14T09:59:15Z" level="info" msg="Unable to check blacklist for frontend-1.insight-d1:_:database, return false"

Thanks, I will be able to "stop" checks during deploy, and to always discard some checks ;)

from consul-alerts.

There's two reason for that message (it should have been a warning, not an info, will fix the logs). Either consul-alerts/config/checks/frontend-1.insight-d1/_/database/blacklisted does not exist, or the value is not true or false. Or maybe a typo? ;)

from consul-alerts.

Yes, the value does not exist (yet). I just noticed that when I tried to debbuging the issue with slack. ;)

from consul-alerts.

haha. yeah. Logging needs better cleanup. I'll add it to my list.

from consul-alerts.

Just release v0.2.0 with changes to how this is implemented. Fixed a possible race condition which @lyrixx pointed out. It's much simpler now.

Check out the README for more info. Cheers.

from consul-alerts.

perfect ;)

from consul-alerts.