Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/slack webhook attachments #423

Closed

Conversation

jcampos8782
Copy link

Builds on an earlier fork to implement the ability to send messages via Slack Webhooks instead of using the Web API. Adds a basic attachment message which specifies the name of the failing check, the target metrics and value of the failing check, and the from and to state. Attachment colors are configurable via various env variables.

@jcampos8782 jcampos8782 force-pushed the feature/slack-webhook-attachments branch from 92cb878 to 41b129e Compare December 16, 2016 08:07
@b-viguier
Copy link

Seems very interesting! We are looking for a better integration of Slack with Seyren, and this PR seems to do the job. Is there any maintainer ready to look at this please?
Thanks

@daneah
Copy link

daneah commented May 30, 2017

Would love to see some movement on this...any interest in getting this merged? @scobal ?

@nick-lange1
Copy link

Any movement on this? Would love to see it merged as the current implementation gets pretty noisy in alerting channels

@daneah
Copy link

daneah commented Jul 30, 2018

@nick-lange1 @b-viguier @jcampos8782 unfortunately it looks like this project is effectively unmaintained (last commit April, 2017). Anyone know of a more active fork of this project?

@b-viguier
Copy link

@daneah For the moment, we just forked the repository and applied modifications proposed in this PR (+ some minor style changes). We intend to move progressively our alerting to Grafana.

@daneah
Copy link

daneah commented Jul 30, 2018

@b-viguier ah! we only recently updated to Grafana v4 so I wasn't aware of this feature. Looks like a great replacement for Seyren—I'll check it out too 😄

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.

4 participants