Ben Abrams
Ben Abrams
I also see that `erubis` dep also called out: https://github.com/sensu-plugins/sensu-plugins-mailer/blob/2.0.0/sensu-plugins-mailer.gemspec#L34
Can you please show me the output of `sudo /opt/sensu/embedded/bin/gem list`?
@stevenviola another report of this, if we can't get this working we will have to revert the change.
I don't honestly see the value in having `contact` and `contacts` I think we should combine them in a breaking change (major versioned) to avoid confusion in the future. You...
Ya it makes total sense to me and I feel the same way about that with handler vs handlers in check definitions. All it does is add confusion. I think...
@stevenviola I just opened a PR to help users with this type of misconfiguration as it is a fairly easy to trap. Once it has been reviewed, merged, and released...
@njomeara can you please try using 2.0.0 which I just released to attempt to catch this misconfiguration?
@njomeara did you see anything in your server logs that would have indicated that there was an issue? The goal was to make it easier to spot the issue rather...
So the second one is working just fine, what is confusing me here is that you are not triggering this: https://github.com/sensu-plugins/sensu-plugins-mailer/blob/2.0.0/bin/handler-mailer.rb#L148-L153 on the first one.
OK I think I figured it out thanks to @cwjohnston who helped me realize whats going on. As you are specifying it from the check as `contact` but in your...