Merge pull request #57 from empovit/upgrade-terraform-1.3

Simplify private_subnets variable with appropriate defaults
equinix · Jun 3, 2024 · f386556 · f386556
2 parents 28441ee + 238312c
commit f386556
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/variables.tf b/variables.tf
@@ -47,19 +47,18 @@ variable "private_subnets" {
   type = list(object({
     name                 = string,
     nat                  = bool,
-    vsphere_service_type = string,
+    vsphere_service_type = optional(string),
     routable             = bool,
     cidr                 = string,
-    reserved_ip_count    = optional(number)
+    reserved_ip_count    = optional(number, 100)
   }))
   default = [
     {
       "name" : "VM Private Net 1",
       "nat" : true,
       "vsphere_service_type" : "management",
       "routable" : true,
-      "cidr" : "172.16.0.0/24",
-      "reserved_ip_count" : 100
+      "cidr" : "172.16.0.0/24"
     },
     {
       "name" : "vMotion",
@@ -91,7 +90,6 @@ variable "public_subnets" {
     {
       "name" : "VM Public Net 1",
       "nat" : false,
-      "vsphere_service_type" : null,
       "routable" : true,
       "ip_count" : 8
     }

diff --git a/versions.tf b/versions.tf
@@ -1,5 +1,6 @@
 terraform {
   required_version = "~> 1.4"
+
   required_providers {
     null = {
       source = "hashicorp/null"
@@ -21,6 +22,7 @@ terraform {
       source = "hashicorp/local"
     }
   }
+
   provider_meta "equinix" {
     module_name = "equinix-metal-vsphere"
   }