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.

Sign up for GitHub

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

Jump to bottom

fix #320 - Kapacitor alerts are less noisy #358

Merged
merged 1 commit into from
Oct 21, 2016
Merged

fix #320 - Kapacitor alerts are less noisy #358

merged 1 commit into from
Oct 21, 2016

Conversation

ndegory
Copy link
Contributor

@ndegory ndegory commented Oct 20, 2016

how to test:

  • export AMP_SLACK_ALERTS_CHANNEL="@you_login" (or #channel)
  • export AMP_SLACK_ALERTS_WEBHOOK_URL=https://hooks.slack.com/services/xxxx/cccc/zzz
  • swarm start
  • monitor your Slack app, notification should appear in the slackbot conversation
  • you should have a few notification, but not a constant flow

subfuzion
subfuzion approved these changes Oct 21, 2016
View reviewed changes
Copy link
Contributor

@subfuzion subfuzion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@subfuzion subfuzion merged commit 59708c9 into master Oct 21, 2016
@subfuzion subfuzion deleted the issue-320 branch October 21, 2016 00:38
@subfuzion subfuzion added this to the 0.2.1 milestone Oct 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@subfuzion subfuzion subfuzion approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.2.1
Development

Successfully merging this pull request may close these issues.
2 participants
@ndegory @subfuzion