From 57ac91949bf9f7597a8470a78901c766258aa353 Mon Sep 17 00:00:00 2001
From: Ash Davies <3853061+DrizzlyOwl@users.noreply.github.com>
Date: Wed, 6 Dec 2023 13:55:00 +0000
Subject: [PATCH] Disable default deployment of Event Hub

* Event Hub namespaces are only useful if we intend on exporting logs out of Azure
---
 terraform/README.md    | 4 ++--
 terraform/variables.tf | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/terraform/README.md b/terraform/README.md
index 98d3bad7d..27fc12fb8 100644
--- a/terraform/README.md
+++ b/terraform/README.md
@@ -186,8 +186,8 @@ If everything looks good, answer `yes` and wait for the new infrastructure to be
 | <a name="input_enable_container_health_probe"></a> [enable\_container\_health\_probe](#input\_enable\_container\_health\_probe) | Enable liveness probes for the Container | `bool` | `true` | no |
 | <a name="input_enable_container_registry"></a> [enable\_container\_registry](#input\_enable\_container\_registry) | Set to true to create a container registry | `bool` | n/a | yes |
 | <a name="input_enable_dns_zone"></a> [enable\_dns\_zone](#input\_enable\_dns\_zone) | Conditionally create a DNS zone | `bool` | n/a | yes |
-| <a name="input_enable_event_hub"></a> [enable\_event\_hub](#input\_enable\_event\_hub) | Send Azure Container App logs to an Event Hub sink | `bool` | n/a | yes |
-| <a name="input_enable_logstash_consumer"></a> [enable\_logstash\_consumer](#input\_enable\_logstash\_consumer) | Create an Event Hub consumer group for Logstash | `bool` | n/a | yes |
+| <a name="input_enable_event_hub"></a> [enable\_event\_hub](#input\_enable\_event\_hub) | Send Azure Container App logs to an Event Hub sink | `bool` | `false` | no |
+| <a name="input_enable_logstash_consumer"></a> [enable\_logstash\_consumer](#input\_enable\_logstash\_consumer) | Create an Event Hub consumer group for Logstash | `bool` | `false` | no |
 | <a name="input_enable_monitoring"></a> [enable\_monitoring](#input\_enable\_monitoring) | Create App Insights monitoring groups for the container app | `bool` | n/a | yes |
 | <a name="input_enable_mssql_database"></a> [enable\_mssql\_database](#input\_enable\_mssql\_database) | Set to true to create an Azure SQL server/database, with a private endpoint within the virtual network | `bool` | n/a | yes |
 | <a name="input_enable_redis_cache"></a> [enable\_redis\_cache](#input\_enable\_redis\_cache) | Set to true to create a Redis Cache | `bool` | n/a | yes |
diff --git a/terraform/variables.tf b/terraform/variables.tf
index e0c55f562..fd1246f7d 100644
--- a/terraform/variables.tf
+++ b/terraform/variables.tf
@@ -141,11 +141,13 @@ variable "monitor_email_receivers" {
 variable "enable_event_hub" {
   description = "Send Azure Container App logs to an Event Hub sink"
   type        = bool
+  default     = false
 }
 
 variable "enable_logstash_consumer" {
   description = "Create an Event Hub consumer group for Logstash"
   type        = bool
+  default     = false
 }
 
 variable "eventhub_export_log_analytics_table_names" {