search

joonty / systemd_mon

Monitor for systemd to alert failed services
MIT License
106 stars 28 forks source link

Add support for hipchat notifier plugin #1

Closed tylerjl closed 9 years ago

tylerjl commented 9 years ago

Hi! First of all, great work on this gem. Very useful tool with a tremendous amount of utility.

I've added support for Hipchat and tested it with my own account and it seems to be working correctly. I've also added examples to the README so this is pretty much just ready to go.

I toyed with the idea of adding the ability to specify an array of users to @-mention in the config yaml but for now I think getting baseline functionality is good enough before expanding the capabilities here.

Cheers!

joonty commented 9 years ago

This is fantastic! It was on my todo list, as I know that hipchat is used widely, but since I don't use it myself never got round to implementing it.

It looks spot on, and I think your decision about keeping it simple for now is exactly right.

Thanks for taking the time to do this, it's really appreciated. I'll give it a test and release a new version of the gem.