-
Notifications
You must be signed in to change notification settings - Fork 148
Support Mattermost via webhooks #25
Comments
@metalmatze we have a new plugin system, which you can do much more, maybe we can create a Prometheus plugin for mm. |
Any updates on this topic? |
@metalmatze sorry no But I was thinking to write the bot using the Mattermost plugin system, but will be another binary and all other configs, not sure if we can keep here or create a new repo for that |
I will add this in my objectives to get this done :) |
Do you have any links for the plugin system? I can also take a look. |
I was thinking to fork your project and rewrite the bot using the plugin mattermost plugin |
I'm happy to have it in this repository, if you want. I don't see any particular reason not too, is there? If you want I can make you a maintainer. |
I did this repo, since the plugin have some specific things :) i used your client and update that to use the v2 of the alertmanager :) https://github.com/cpanato/mattermost-plugin-alertmanager if you want i can convert your project to use the v2 as well |
Hmm, I'd be fine with having it in here. The Telegram package in pkg/telegram should be standalone as well and doesn't rely on any other package. The only dependencies for those packages are the alertmanager-bot/cmd/alertmanager-bot/main.go Line 190 in 989c04b
alertmanager-bot/cmd/alertmanager-bot/main.go Line 165 in 989c04b
Everything else needs to be implemented by the package itself with total control/freedom/responsibilities. 😉 There's already a PR pending with a generated API v2 client #47. |
We have a prometheus/alertmanager running inside a walled garden, but outside of our k8s cluster and a cloud hosted mattermost server. would your mattermost integrations work to send alerts for that? |
Right now my bot still doesn't support Mattermost. Although I started with the overhaul and simply don't have the time for this project to make a full rewrite on my own. |
Thus, I still recommend using the one my friend @cpanato has built! :) |
Following the discussion going on at prometheus/alertmanager#1512.
We should try to support Mattermost next to Telegram.
We can first start by supporting message delivering to Mattermost. Afterwards we can look into supporting mattermost commands for the bot.
cc @cpanato @mxinden
The text was updated successfully, but these errors were encountered: