feat: Supporting no static outbound ips as well

Author: dploeger

main.tf

@@ -48,10 +48,13 @@ resource "azurerm_kubernetes_cluster" "k8s" {
     network_plugin    = "azure"
     network_policy    = var.network_policy
     load_balancer_sku = length(var.node_pools) > 0 ? "Standard" : var.load_balancer_sku
-    load_balancer_profile {
-      outbound_ip_address_ids  = azurerm_public_ip.public-ip-outbound[*].id
-      outbound_ports_allocated = var.outbound_ports_allocated
-      idle_timeout_in_minutes  = var.idle_timeout
+    dynamic load_balancer_profile {
+      for_each = azurerm_public_ip.public-ip-outbound
+      content {
+        outbound_ip_address_ids  = azurerm_public_ip.public-ip-outbound[*].id
+        outbound_ports_allocated = var.outbound_ports_allocated
+        idle_timeout_in_minutes  = var.idle_timeout
+      }
     }
   }
 
@@ -88,4 +91,4 @@ resource "azurerm_public_ip" "public-ip-outbound" {
   location            = var.location
   resource_group_name = var.resource_group
   sku                 = "Standard"
-}
+}

vars.tf

@@ -129,10 +129,10 @@ variable "static_outbound_ip_count" {
     bottlenecks. Recommended in that case is to set the count at least +5 more than the count of kubernetes nodes.
   EOF
   validation {
-    condition     = var.static_outbound_ip_count >= 1 && var.static_outbound_ip_count <= 100
-    error_message = "Static_outbound_ip_count has to be between 1 and 100 including."
+    condition     = var.static_outbound_ip_count >= 0 && var.static_outbound_ip_count <= 100
+    error_message = "Static_outbound_ip_count has to be between 0 and 100 including."
   }
-  default = 1
+  default = 0
 }
 
 variable "outbound_ports_allocated" {