benjojo / alertmanager-discord

Take your alertmanager alerts, into discord
Apache License 2.0
177 stars 78 forks source link

Upstream this #22

Closed roidelapluie closed 2 years ago

roidelapluie commented 3 years ago

Hello,

We have lifted the moratorium on integrations in alertmanager.

Would you be willing to help us bring discord as an official native integration?

Thanks for your answer.

benjojo commented 3 years ago

Sure! How is best to work with the project? Feel free to redirect this convo to email (in my profile) if that helps

roidelapluie commented 3 years ago

I invite you to open a pull request within the alertmanager repo. We host weekly contributors office hours if you need help : https://prometheus.io/community :)

AlexH0661 commented 3 years ago

Hello, I was wondering if there has been any movement on this. It would be great to see Discord webhooks natively handled in AlertManager

benjojo commented 3 years ago

I have made a branch to test it out. But I've been busy with other stuff the past while. On the bright side, I have some holiday coming up, so expect a PR "soon" :)

On Sun, 11 Jul 2021, 12:01 AlexH0661, @.***> wrote:

Hello, I was wondering if there has been any movement on this. It would be great to see Discord webhooks natively handled in AlertManager

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/benjojo/alertmanager-discord/issues/22#issuecomment-877780014, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALPK4SRZDVWJGAGIJNRVTLTXF2W7ANCNFSM44LBEG2Q .

AlexH0661 commented 3 years ago

Nice work :)That's fair enough!I'll keep a look out :)On 11 Jul 2021 22:14, Ben Cox @.***> wrote:

I have made a branch to test it out. But I've been busy with other stuff

the past while. On the bright side, I have some holiday coming up, so

expect a PR "soon" :)

On Sun, 11 Jul 2021, 12:01 AlexH0661, @.***> wrote:

Hello,

I was wondering if there has been any movement on this.

It would be great to see Discord webhooks natively handled in AlertManager

You are receiving this because you commented.

Reply to this email directly, view it on GitHub

https://github.com/benjojo/alertmanager-discord/issues/22#issuecomment-877780014,

or unsubscribe

https://github.com/notifications/unsubscribe-auth/AALPK4SRZDVWJGAGIJNRVTLTXF2W7ANCNFSM44LBEG2Q

.

—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or unsubscribe.

Lusitaniae commented 2 years ago

friendly reminder, if there's anything left to do please let us know if we can help

benjojo commented 2 years ago

ack, I have my WIP branch kicking around somewhere, I'll see if I can finish it in the next 2 weeks

metalmatze commented 2 years ago

Hey. In the last week I had to integrate Discord alerting for one of our projects and tried this Docker container. It didn't work so instead of further debugging the container itself, I went ahead and integrated Discord into alertmanager directly. This work was heavily inspired by your work on this repository.

Here's the PR: https://github.com/prometheus/alertmanager/pull/2948