From 1dc413c5fc01890d0e216ea3f783f6decb02a198 Mon Sep 17 00:00:00 2001
From: Cody Fletcher <cody@codyfletcher.com>
Date: Mon, 23 Jan 2023 15:30:05 -0800
Subject: [PATCH] fix: Give unique alarm_name to avoid collision when default
 and gunicorn both create the same alarm name.

---
 modules/internal/autoscaling/main.tf | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/modules/internal/autoscaling/main.tf b/modules/internal/autoscaling/main.tf
index a48adcf..0d7b213 100644
--- a/modules/internal/autoscaling/main.tf
+++ b/modules/internal/autoscaling/main.tf
@@ -50,7 +50,7 @@ resource "aws_appautoscaling_policy" "down" {
 
 # CloudWatch alarm that triggers the autoscaling up policy
 resource "aws_cloudwatch_metric_alarm" "service_cpu_high" {
-  alarm_name          = "${terraform.workspace}_cpu_utilization_high"
+  alarm_name          = "${var.service_name}_cpu_utilization_high"
   comparison_operator = "GreaterThanOrEqualToThreshold"
   evaluation_periods  = "2"
   metric_name         = "CPUUtilization"
@@ -69,7 +69,7 @@ resource "aws_cloudwatch_metric_alarm" "service_cpu_high" {
 
 # CloudWatch alarm that triggers the autoscaling down policy
 resource "aws_cloudwatch_metric_alarm" "service_cpu_low" {
-  alarm_name          = "${terraform.workspace}_cpu_utilization_low"
+  alarm_name          = "${var.service_name}_cpu_utilization_low"
   comparison_operator = "LessThanOrEqualToThreshold"
   evaluation_periods  = "2"
   metric_name         = "CPUUtilization"