Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zabbix Monitoring Server Integration. #21

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

Conversation

tonyborries
Copy link

Use Zabbix Trigger Status and Severity to override the color of messages to HipChat.

Signed-off-by: TonyB [email protected]

Use Zabbix Trigger Status and Severity to override the color of messages to HipChat.

Signed-off-by: TonyB <[email protected]>
@gergelyke
Copy link

thanks!

@jimsmith
Copy link
Contributor

jimsmith commented Mar 7, 2017

Why has this not been merged in ? this is a highly needed addition

@jimsmith
Copy link
Contributor

Any update this is required as a user use case as what the original poster submitted I don't want to re-invent the wheel again.
Thanks.

STATUS=$(echo $STATUS | tr '[:lower:]' '[:upper:]')
if [[ "$STATUS" == 'OK' ]]; then
COLOR="green";
elif [[ $LEVEL == 'DISASTER' ]]; then
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should these be $STATUS instead of $LEVEL?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't have a running Zabbix instance to test against anymore to verify this, however I believe the PR is correct as it exists. The Zabbix Status is either "OK" or "PROBLEM", and if it is "PROBLEM" then Level is used to map the severity.

@rberrelleza
Copy link
Contributor

@jimsmith There seem to be issues with the PR (I'm no expert on Zabbix though), so can't merge it yet. Could you take a look? Not sure if the original submitter is still around to correct it. If not, maybe you can take over?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants