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

Add ability to acknowledge with an expiration time for icinga hosts, vs ... #2

Merged
merged 2 commits into from
Jul 2, 2014

Conversation

deeno35
Copy link
Contributor

@deeno35 deeno35 commented Jun 30, 2014

...always acknowledging indefinitely. Default behavior is unchanged (ack forever).

…vs always acknowledging indefinitely. Default behavior is unchanged (ack forever).
@itsjohannawren
Copy link
Owner

David, PagerDuty has an acknowledgement timeout already. Is having Icinga also timeout necessary? If so, will Icinga fire a notification when the ack times-out?

@itsjohannawren itsjohannawren self-assigned this Jul 1, 2014
@deeno35
Copy link
Contributor Author

deeno35 commented Jul 1, 2014

The usability problem right now is that setting an ACK in PD, acks nagios/icinga forever. If somebody subsequently resolves in PD (resolve all, accidental resolve, etc), I'll never receive another alert for that service unless it changes state. It's probably only an issue for me since I'm not using PD as my dashboard - I'm just using it as a paging service and I still go back to nagios/icinga for source of truth.

When the acknowledgement expires in icinga, it's the same as when a downtime expires in nagios. Icinga will go back onto continuous alerts based on notification interval and timeperiod for that check.

You do raise a good point though. I wonder if my customization is really only useful for how I'm using PD.

@itsjohannawren
Copy link
Owner

That's an interesting point. I think it could worth putting it in.

@itsjohannawren itsjohannawren merged commit 10a3c17 into itsjohannawren:master Jul 2, 2014
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.

2 participants