Merge pull request #173 from wso2/update-public-ip-zones-1

Add configurable public_ip_zones variable for Public IP availability zones

Author: ayeshajay

modules/azurerm/Firewall-Multiple-PIP/public_ip.tf

@@ -14,7 +14,7 @@ resource "azurerm_public_ip_prefix" "public_ip_prefix_fw" {
   location            = var.location
   resource_group_name = var.resource_group_name
   prefix_length       = var.public_ip_prefix_length
-  zones               = [1, 2, 3]
+  zones               = var.zones
   tags                = var.tags
 }
 
@@ -26,7 +26,7 @@ resource "azurerm_public_ip" "firewall_public_ip" {
   allocation_method   = "Static"
   sku                 = "Standard"
   public_ip_prefix_id = null == var.dynamic_nat_rules[each.key].public_ip_prefix_id ? azurerm_public_ip_prefix.public_ip_prefix_fw.id : var.dynamic_nat_rules[each.key].public_ip_prefix_id
-  zones               = [1, 2, 3]
+  zones               = var.zones
   tags                = var.tags
   depends_on = [
     azurerm_public_ip_prefix.public_ip_prefix_fw

modules/azurerm/Firewall-Multiple-PIP/variables.tf

@@ -139,3 +139,9 @@ variable "private_endpoint_network_policies" {
   description = "Enable or Disable network policies for the private endpoint on the subnet. Possible values are Disabled, Enabled, NetworkSecurityGroupEnabled and RouteTableEnabled"
   type        = string
 }
+
+variable "zones" {
+  default     = [1, 2, 3]
+  description = "A collection containing the availability zone to allocate the Public IP in"
+  type        = list(number)
+}