Merge pull request #77 from provectus/DQ-123

Dq 123
provectus · May 18, 2023 · 7297786 · 7297786
2 parents a613cac + d11fb25
commit 7297786
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/terraform/lambda_push_report.tf b/terraform/lambda_push_report.tf
@@ -40,7 +40,7 @@ module "data_reports_alerting" {
   slack_channel     = var.data_reports_notification_settings.channel
   slack_webhook_url = var.data_reports_notification_settings.webhook_url
 
-  slack_sns_topic_name = "dqg-data_reports"
+  slack_sns_topic_name = "dqg-data_reports-${var.environment}"
   slack_username       = "DQG-alerting"
 
   resource_name_prefix = local.resource_name_prefix

diff --git a/terraform/modules/alerting/main.tf b/terraform/modules/alerting/main.tf
@@ -9,7 +9,7 @@ resource "aws_kms_ciphertext" "slack_url" {
 
 module "slack_notification" {
   source  = "terraform-aws-modules/notify-slack/aws"
-  version = "~> 4.0"
+  version = "~> 5.0"
 
   sns_topic_name = var.slack_sns_topic_name