Skip to content
This repository has been archived by the owner on Dec 6, 2022. It is now read-only.

Support for Icinga #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Support for Icinga #2

wants to merge 4 commits into from

Conversation

elmer
Copy link
Contributor

@elmer elmer commented Jan 21, 2011

It would be nice if Angelia can be used when using Icinga instead of Nagios. Thanks

When using mailto://, Angelia is unable to send the notification due
to 'block not supplied' exception. Mail.defaults expects a block.
This patch allows you to use Angelia when calling from Icinga. When
--icinga is pass to angelia-nagios-send, it gathers the ICINGA_*
environment variables instead.

Use it as:

define command {
  command_name notify-by-angelia
  command_line /usr/sbin/angelia-nagios-send --icinga -c /etc/angelia/angelia.cfg --service-notify -r $CONTACTEMAILS$
}
@ripienaar
Copy link
Owner

Could we make this easier by just checking if there are any vars set with ^ICINGA or ^NAGIOS in the ENV and detecting which backend we use? perhaps there's one that always get set and so would be a good indicator?

Just a thought but then we dont need to add the extra flag which would be a win

as for some reason "uninitialized constant Angelia::Plugin::Clickatell::API, will retry" is being raised.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants