Feature Request - Add PagerDuty as a notification target

ndelitski / rancher-alarms

Will kick your ass if found unhealthy service in Rancher environment

85 stars 20 forks source link

Open jmreicha opened 7 years ago

jmreicha commented 7 years ago

It would be useful to send alerts for downed services to PagerDuty. I think it should only require a webhook URL.

VAdamec commented 7 years ago

Did you try it ? seems to be easy, also PagerDuty provides examples for javascript.

jmreicha commented 7 years ago

@VAdamec I don't really follow. You mean, use the PD webhook in place of something or add some code to the project to allow for PD webhooks?

VAdamec commented 7 years ago

Feature requests are fine, but what about to contribute :-)

jmreicha commented 7 years ago

Oh yeah, I think I will try to add it this week and send a PR :)

VAdamec commented 7 years ago

Hm, we could take https://www.npmjs.com/package/pagerduty-alert it will be quite easy. With Axios we need to calculate auth header (https://www.pagerduty.com/blog/pagerduty-javascript-integration/) that seems to be also easy (with API Token Authentication).

flaccid commented 6 years ago

Any updates @ndelitski? :)