Skip to content
This repository has been archived by the owner on Aug 14, 2022. It is now read-only.

Support Mattermost via webhooks #25

Open
metalmatze opened this issue Aug 13, 2018 · 14 comments
Open

Support Mattermost via webhooks #25

metalmatze opened this issue Aug 13, 2018 · 14 comments

Comments

@metalmatze
Copy link
Owner

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

@cpanato
Copy link

cpanato commented Aug 27, 2018

@metalmatze we have a new plugin system, which you can do much more, maybe we can create a Prometheus plugin for mm.
I will spend some time here to se how it works and see where I can contribute :) thanks!!

@metalmatze
Copy link
Owner Author

Any updates on this topic?

@cpanato
Copy link

cpanato commented Nov 8, 2018

@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

@cpanato
Copy link

cpanato commented Nov 8, 2018

I will add this in my objectives to get this done :)

@metalmatze
Copy link
Owner Author

Do you have any links for the plugin system? I can also take a look.

@cpanato
Copy link

cpanato commented Nov 8, 2018

@cpanato
Copy link

cpanato commented Feb 27, 2019

I was thinking to fork your project and rewrite the bot using the plugin mattermost plugin

@cpanato
Copy link

cpanato commented Mar 24, 2019

the webhook notification is working
Screenshot 2019-03-24 at 21 50 04
I will use the same approach you did and will use your pkg/alertmanager for the functions, but i'm implemting this as a mattermost plugin 😄

@metalmatze
Copy link
Owner Author

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.

@cpanato
Copy link

cpanato commented Apr 5, 2019

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

@metalmatze
Copy link
Owner Author

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 same applies to a package for Hangouts Chat that @mulbc wants to contribute.

The only dependencies for those packages are the chan notify.WebhookMessage and store.

webhooks := make(chan notify.WebhookMessage, 32)

var kvStore store.Store

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.

@jdbarnes-isi
Copy link

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?

@metalmatze
Copy link
Owner Author

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.

@metalmatze
Copy link
Owner Author

Thus, I still recommend using the one my friend @cpanato has built! :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants