Skip to content

Commit

Permalink
BFD-3412: Change instances of #bfd-test slack channel to #bfd-interna…
Browse files Browse the repository at this point in the history
…l-alerts (#2352)
  • Loading branch information
dondevun authored Jun 24, 2024
1 parent 78c6f78 commit bb2b96f
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ resource "aws_sns_topic_policy" "cloudwatch_alarms_slack_bfd_notices" {

resource "aws_sns_topic" "cloudwatch_alarms_slack_bfd_test" {
name = "bfd-${local.env}-cloudwatch-alarms-slack-bfd-test"
display_name = "BFD Cloudwatch Alarms alerts to #bfd-test. Created by Terraform."
display_name = "BFD Cloudwatch Alarms alerts to #bfd-internal-alerts. Created by Terraform."
kms_master_key_id = local.kms_master_key_id
}

Expand Down
2 changes: 1 addition & 1 deletion ops/terraform/services/pipeline/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ locals {
ok_actions = local.is_prod ? [data.aws_sns_topic.ok[0].arn] : []

# The log availability alarm will post an incident in prod; in other envs it will get posted
# to #bfd-test
# to #bfd-internal-alerts
# TODO: Replace testing SNS topic in BFD-2244
log_availability_alarm_actions = local.is_ephemeral_env ? [] : local.is_prod ? [data.aws_sns_topic.bfd_notices_slack_alarm[0].arn] : [data.aws_sns_topic.bfd_test_slack_alarm[0].arn]

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,5 +66,5 @@ def _solid_cluster(
" query"
)
)
>> Slack("#bfd-test")
>> Slack("#bfd-internal-alerts")
)

0 comments on commit bb2b96f

Please sign in to comment.