Skip to content

feat: monitoring alarms to slack#158

Merged
shumkov merged 5 commits into
v0.13-devfrom
monitoring_alarms_to_slack
May 20, 2020
Merged

feat: monitoring alarms to slack#158
shumkov merged 5 commits into
v0.13-devfrom
monitoring_alarms_to_slack

Conversation

@v2nek
Copy link
Copy Markdown
Contributor

@v2nek v2nek commented May 19, 2020

This patch adds possibility to send alarms from monitoring to sns topic

@v2nek v2nek requested a review from shumkov May 19, 2020 12:40
@v2nek v2nek changed the base branch from v0.13-dev to master May 19, 2020 12:40
@shumkov shumkov changed the title feat: Monitoring alarms to slack feat: monitoring alarms to slack May 19, 2020
Comment thread terraform/aws/variables.tf Outdated
default = ""
}

variable "sns_arn" {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

montitoring_sns_arn ?

Comment thread terraform/aws/variables.tf Outdated
default = ""
}


Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Remove extra line

@shumkov shumkov changed the base branch from master to v0.13-dev May 19, 2020 15:01
@shumkov shumkov added this to the v0.13 milestone May 19, 2020
@shumkov shumkov changed the base branch from v0.13-dev to master May 20, 2020 16:22
@shumkov shumkov changed the base branch from master to v0.13-dev May 20, 2020 16:22
Copy link
Copy Markdown
Collaborator

@shumkov shumkov left a comment

Choose a reason for hiding this comment

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

👍

@shumkov shumkov merged commit 3da53dd into v0.13-dev May 20, 2020
@shumkov shumkov deleted the monitoring_alarms_to_slack branch May 20, 2020 16:24
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.

2 participants