Skip to content

Use #send_notice as a way to 'ack' in Pagerduty service.#3

Open
asenchi wants to merge 2 commits intomasterfrom
pagerduty-ack
Open

Use #send_notice as a way to 'ack' in Pagerduty service.#3
asenchi wants to merge 2 commits intomasterfrom
pagerduty-ack

Conversation

@asenchi
Copy link
Owner

@asenchi asenchi commented Jun 19, 2014

Not sure how this hasn't been in here before, seems required.

@asenchi
Copy link
Owner Author

asenchi commented Jun 19, 2014

Hmm, so this might be breaking... going to check with some people I know using this.

@gorsuch
Copy link
Contributor

gorsuch commented Jun 19, 2014

Curious - why do you want comsat to be able to ack a page?

@asenchi
Copy link
Owner Author

asenchi commented Jun 19, 2014

That's a valid question. Two things I can think of:

  • Service handlers ack'ing pages and resolving upon success (I need to think of this a bit more)
  • Frontends where alerts can be managed.

I'm not sure... at first it seemed incredibly useful, but I think you're question is a good one. I do know that I think moving to aliasing notice to alert rather than the other way is a better interface. Need to think on it a bit...

@gorsuch
Copy link
Contributor

gorsuch commented Jun 19, 2014

Service handlers ack'ing pages and resolving upon success

Oh, interesting. So:

  • monitoring system has an embedded comsat that it uses to route alerts to an event handler
  • event handler has an embedded comsat thing as well, allowing it to route messages as well?

Repository owner deleted a comment from erenakkaya Feb 23, 2024
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