fix: using list of object type for the new variable

Author: pawan1210

examples/internal-lb-cloud-run/main.tf

@@ -116,14 +116,14 @@ module "internal-lb-http-frontend" {
   url_map_input         = module.internal-lb-http-backend.backend_service_info
   network               = module.internal-lb-network.network_name
   load_balancing_scheme = "INTERNAL_MANAGED"
-  internal_forwarding_rule_configs = {
-    "1" : {
+  internal_forwarding_rules_config = [
+    {
       "subnetwork" : module.internal-lb-subnet.subnets["us-east1/int-lb-subnet-a"].id
     },
-    "2" : {
+    {
       "subnetwork" : module.internal-lb-subnet.subnets["us-south1/int-lb-subnet-b"].id
     }
-  }
+  ]
 }
 
 resource "google_vpc_access_connector" "internal_lb_vpc_connector" {

modules/frontend/README.md

@@ -19,7 +19,7 @@ This module creates `HTTP(S) forwarding rule` and its dependencies. This modules
 | http\_port | The port for the HTTP load balancer | `number` | `80` | no |
 | https\_port | The port for the HTTPS load balancer | `number` | `443` | no |
 | https\_redirect | Set to `true` to enable https redirect on the lb. | `bool` | `false` | no |
-| internal\_forwarding\_rule\_configs | Map of internal managed forwarding rule configs. One of 'address' or 'subnetwork' is required for each. | <pre>map(object({<br>    address    = optional(string)<br>    subnetwork = optional(string)<br>  }))</pre> | `{}` | no |
+| internal\_forwarding\_rules\_config | List of internal managed forwarding rules config. One of 'address' or 'subnetwork' is required for each. | <pre>list(object({<br>    address    = optional(string)<br>    subnetwork = optional(string)<br>  }))</pre> | `[]` | no |
 | ipv6\_address | An existing IPv6 address to use (the actual IP address value) | `string` | `null` | no |
 | labels | The labels to attach to resources created by this module | `map(string)` | `{}` | no |
 | load\_balancing\_scheme | Load balancing scheme type (EXTERNAL for classic external load balancer, EXTERNAL\_MANAGED for Envoy-based load balancer, and INTERNAL\_SELF\_MANAGED for traffic director) | `string` | `"EXTERNAL_MANAGED"` | no |

modules/frontend/main.tf

@@ -58,7 +58,9 @@ resource "google_compute_global_forwarding_rule" "http" {
 }
 
 resource "google_compute_global_forwarding_rule" "internal_managed_http" {
-  for_each = local.create_http_forward && local.is_internal_managed ? var.internal_forwarding_rule_configs : {}
+  for_each = local.create_http_forward && local.is_internal_managed ? {
+    for index, config in var.internal_forwarding_rules_config : index => config
+  } : {}
 
   provider              = google-beta
   project               = var.project_id
@@ -86,7 +88,9 @@ resource "google_compute_global_forwarding_rule" "https" {
 }
 
 resource "google_compute_global_forwarding_rule" "internal_managed_https" {
-  for_each = var.ssl && local.is_internal_managed ? var.internal_forwarding_rule_configs : {}
+  for_each = var.ssl && local.is_internal_managed ? {
+    for index, config in var.internal_forwarding_rules_config : index => config
+  } : {}
 
   provider              = google-beta
   project               = var.project_id
@@ -125,7 +129,9 @@ resource "google_compute_global_forwarding_rule" "http_ipv6" {
 }
 
 resource "google_compute_global_forwarding_rule" "internal_managed_http_ipv6" {
-  for_each = var.enable_ipv6 && local.create_http_forward && local.is_internal_managed ? var.internal_forwarding_rule_configs : {}
+  for_each = var.enable_ipv6 && local.create_http_forward && local.is_internal_managed ? {
+    for index, config in var.internal_forwarding_rules_config : index => config
+  } : {}
 
   provider              = google-beta
   project               = var.project_id
@@ -152,7 +158,9 @@ resource "google_compute_global_forwarding_rule" "https_ipv6" {
 }
 
 resource "google_compute_global_forwarding_rule" "internal_managed_https_ipv6" {
-  for_each = var.enable_ipv6 && var.ssl && local.is_internal_managed ? var.internal_forwarding_rule_configs : {}
+  for_each = var.enable_ipv6 && var.ssl && local.is_internal_managed ? {
+    for index, config in var.internal_forwarding_rules_config : index => config
+  } : {}
 
   provider              = google-beta
   project               = var.project_id

modules/frontend/metadata.yaml

@@ -187,14 +187,14 @@ spec:
       - name: http_keep_alive_timeout_sec
         description: Specifies how long to keep a connection open, after completing a response, while there is no matching traffic (in seconds).
         varType: number
-      - name: internal_forwarding_rule_configs
-        description: Map of internal managed forwarding rule configs. One of 'address' or 'subnetwork' is required for each.
+      - name: internal_forwarding_rules_config
+        description: List of internal managed forwarding rules config. One of 'address' or 'subnetwork' is required for each.
         varType: |-
-          map(object({
+          list(object({
               address    = optional(string)
               subnetwork = optional(string)
             }))
-        defaultValue: {}
+        defaultValue: []
     outputs:
       - name: external_ip
         description: The external IPv4 assigned to the global fowarding rule.

modules/frontend/variables.tf

@@ -199,11 +199,11 @@ variable "http_keep_alive_timeout_sec" {
   default     = null
 }
 
-variable "internal_forwarding_rule_configs" {
-  description = "Map of internal managed forwarding rule configs. One of 'address' or 'subnetwork' is required for each."
-  type = map(object({
+variable "internal_forwarding_rules_config" {
+  description = "List of internal managed forwarding rules config. One of 'address' or 'subnetwork' is required for each."
+  type = list(object({
     address    = optional(string)
     subnetwork = optional(string)
   }))
-  default = {}
+  default = []
 }