Merge pull request #274 from silinternational/rm-cloudflare-domain

remove cloudflare_domain from 010-cluster
silinternational · Nov 12, 2024 · 930a8d1 · 930a8d1
2 parents cf45ae2 + 598b2d5
commit 930a8d1
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/terraform/010-cluster/main.tf b/terraform/010-cluster/main.tf
@@ -114,7 +114,7 @@ resource "aws_cloudwatch_log_group" "logs" {
  * Create CloudWatch Dashboard for services that will be in this cluster
  */
 module "ecs-service-cloudwatch-dashboard" {
-  count = var.create_dashboard && var.cloudflare_domain != "" ? 1 : 0
+  count = var.create_dashboard ? 1 : 0
 
   source  = "silinternational/ecs-service-cloudwatch-dashboard/aws"
   version = "~> 3.1"

diff --git a/terraform/010-cluster/vars.tf b/terraform/010-cluster/vars.tf
@@ -21,12 +21,6 @@ variable "cert_domain_name" {
   type = string
 }
 
-variable "cloudflare_domain" {
-  description = "The base domain name to be used for Cloudflare resources, e.g. example.net"
-  type        = string
-  default     = ""
-}
-
 variable "create_dashboard" {
   description = "Set to false to remove the Cloudwatch Dashboard"
   type        = bool