fix(deps): update terraform-module <br> * There will be no -vpc suffix in the VPC name (#909)

terraform-ibm-modules-ops · web-flow · commit d9bedcc6566a · 2025-04-11T13:02:40.000+05:30
diff --git a/README.md b/README.md
@@ -851,17 +851,17 @@ module "cluster_pattern" {
 
 | Name | Source | Version |
 |------|--------|---------|
-| <a name="module_bastion_host"></a> [bastion\_host](#module\_bastion\_host) | terraform-ibm-modules/landing-zone-vsi/ibm | 4.4.0 |
-| <a name="module_cluster"></a> [cluster](#module\_cluster) | terraform-ibm-modules/base-ocp-vpc/ibm | 3.41.7 |
+| <a name="module_bastion_host"></a> [bastion\_host](#module\_bastion\_host) | terraform-ibm-modules/landing-zone-vsi/ibm | 4.6.0 |
+| <a name="module_cluster"></a> [cluster](#module\_cluster) | terraform-ibm-modules/base-ocp-vpc/ibm | 3.44.0 |
 | <a name="module_dynamic_values"></a> [dynamic\_values](#module\_dynamic\_values) | ./dynamic_values | n/a |
-| <a name="module_f5_vsi"></a> [f5\_vsi](#module\_f5\_vsi) | terraform-ibm-modules/landing-zone-vsi/ibm | 4.4.0 |
+| <a name="module_f5_vsi"></a> [f5\_vsi](#module\_f5\_vsi) | terraform-ibm-modules/landing-zone-vsi/ibm | 4.6.0 |
 | <a name="module_key_management"></a> [key\_management](#module\_key\_management) | ./kms | n/a |
 | <a name="module_placement_group_map"></a> [placement\_group\_map](#module\_placement\_group\_map) | ./dynamic_values/config_modules/list_to_map | n/a |
 | <a name="module_ssh_keys"></a> [ssh\_keys](#module\_ssh\_keys) | ./ssh_key | n/a |
 | <a name="module_teleport_config"></a> [teleport\_config](#module\_teleport\_config) | ./teleport_config | n/a |
-| <a name="module_update_cbr_vpc_zone"></a> [update\_cbr\_vpc\_zone](#module\_update\_cbr\_vpc\_zone) | terraform-ibm-modules/cbr/ibm//modules/cbr-zone-module | 1.27.0 |
-| <a name="module_vpc"></a> [vpc](#module\_vpc) | terraform-ibm-modules/landing-zone-vpc/ibm | 7.20.1 |
-| <a name="module_vsi"></a> [vsi](#module\_vsi) | terraform-ibm-modules/landing-zone-vsi/ibm | 4.4.0 |
+| <a name="module_update_cbr_vpc_zone"></a> [update\_cbr\_vpc\_zone](#module\_update\_cbr\_vpc\_zone) | terraform-ibm-modules/cbr/ibm//modules/cbr-zone-module | 1.29.0 |
+| <a name="module_vpc"></a> [vpc](#module\_vpc) | terraform-ibm-modules/landing-zone-vpc/ibm | 7.22.8 |
+| <a name="module_vsi"></a> [vsi](#module\_vsi) | terraform-ibm-modules/landing-zone-vsi/ibm | 4.6.0 |
 
 ### Resources
 
diff --git a/bastion_host.tf b/bastion_host.tf
@@ -42,7 +42,7 @@ module "teleport_config" {
 
 module "bastion_host" {
   source                          = "terraform-ibm-modules/landing-zone-vsi/ibm"
-  version                         = "4.4.0"
+  version                         = "4.6.0"
   for_each                        = local.bastion_vsi_map
   resource_group_id               = each.value.resource_group == null ? null : local.resource_groups[each.value.resource_group]
   create_security_group           = each.value.security_group == null ? false : true
diff --git a/cbr.tf b/cbr.tf
@@ -3,7 +3,7 @@
 ##############################################################################
 module "update_cbr_vpc_zone" {
   source                = "terraform-ibm-modules/cbr/ibm//modules/cbr-zone-module"
-  version               = "1.27.0"
+  version               = "1.29.0"
   count                 = var.existing_vpc_cbr_zone_id != null ? 1 : 0
   use_existing_cbr_zone = true
   existing_zone_id      = var.existing_vpc_cbr_zone_id
diff --git a/cluster.tf b/cluster.tf
@@ -244,7 +244,7 @@ module "cluster" {
     if cluster.kube_type == "openshift"
   }
   source             = "terraform-ibm-modules/base-ocp-vpc/ibm"
-  version            = "3.41.7"
+  version            = "3.44.0"
   resource_group_id  = local.resource_groups[each.value.resource_group]
   region             = var.region
   cluster_name       = each.value.cluster_name
diff --git a/f5_vsi.tf b/f5_vsi.tf
@@ -117,7 +117,7 @@ locals {
 
 module "f5_vsi" {
   source                        = "terraform-ibm-modules/landing-zone-vsi/ibm"
-  version                       = "4.4.0"
+  version                       = "4.6.0"
   for_each                      = local.f5_vsi_map
   resource_group_id             = each.value.resource_group == null ? null : local.resource_groups[each.value.resource_group]
   create_security_group         = each.value.security_group == null ? false : true
diff --git a/main.tf b/main.tf
@@ -33,7 +33,7 @@ locals {
 # Due to existing implicit dependencies we do not think this will be an issue, including auth policies for activity tracker.
 module "vpc" {
   source                      = "terraform-ibm-modules/landing-zone-vpc/ibm"
-  version                     = "7.20.1"
+  version                     = "7.22.8"
   for_each                    = local.vpc_map
   name                        = each.value.prefix
   existing_vpc_id             = each.value.existing_vpc_id
diff --git a/patterns/vsi-extension/main.tf b/patterns/vsi-extension/main.tf
@@ -45,7 +45,7 @@ locals {
 
 module "vsi" {
   source                          = "terraform-ibm-modules/landing-zone-vsi/ibm"
-  version                         = "4.4.0"
+  version                         = "4.6.0"
   resource_group_id               = data.ibm_is_vpc.vpc_by_id.resource_group
   create_security_group           = true
   prefix                          = "${var.prefix}-vsi"
diff --git a/tests/pr_test.go b/tests/pr_test.go
@@ -443,9 +443,9 @@ func setupOptionsVsiExtention(t *testing.T, prefix string, region string, existi
 		fmt.Println(err)
 		return nil
 	}
-	// Loop through the vpcs and find the vpc_id when vpc_name is "<prefix>-management-vpc"
+	// Loop through the vpcs and find the vpc_id when vpc_name is "<prefix>-management"
 	for _, vpc := range vpcs {
-		if vpc.VpcName == fmt.Sprintf("%s-management-vpc", prefix) {
+		if vpc.VpcName == fmt.Sprintf("%s-management", prefix) {
 			managementVpcID = vpc.VpcID
 		}
 	}
diff --git a/tests/resources/slz-vpc/outputs.tf b/tests/resources/slz-vpc/outputs.tf
@@ -9,15 +9,15 @@ output "prefix" {
 
 output "management_vpc_id" {
   value = lookup(
-    [for vpc in module.landing_zone.vpc_data : vpc if vpc.vpc_name == "${var.prefix}-management-vpc"][0],
+    [for vpc in module.landing_zone.vpc_data : vpc if vpc.vpc_name == "${var.prefix}-management"][0],
     "vpc_id",
   "")
   description = "Management VPC ID"
 }
 
 output "workload_vpc_id" {
   value = lookup(
-    [for vpc in module.landing_zone.vpc_data : vpc if vpc.vpc_name == "${var.prefix}-workload-vpc"][0],
+    [for vpc in module.landing_zone.vpc_data : vpc if vpc.vpc_name == "${var.prefix}-workload"][0],
     "vpc_id",
   "")
   description = "Workload VPC ID"
diff --git a/virtual_servers.tf b/virtual_servers.tf
@@ -41,7 +41,7 @@ data "ibm_is_image" "image" {
 
 module "vsi" {
   source                          = "terraform-ibm-modules/landing-zone-vsi/ibm"
-  version                         = "4.4.0"
+  version                         = "4.6.0"
   for_each                        = local.vsi_map
   resource_group_id               = each.value.resource_group == null ? null : local.resource_groups[each.value.resource_group]
   create_security_group           = each.value.security_group == null ? false : true

Original file line number	Diff line number	Diff line change
`@@ -244,7 +244,7 @@ module "cluster" {`
`244`	`244`	`if cluster.kube_type == "openshift"`
`245`	`245`	`}`
`246`	`246`	`source = "terraform-ibm-modules/base-ocp-vpc/ibm"`
`247`		`- version = "3.41.7"`
	`247`	`+ version = "3.44.0"`
`248`	`248`	`resource_group_id = local.resource_groups[each.value.resource_group]`
`249`	`249`	`region = var.region`
`250`	`250`	`cluster_name = each.value.cluster_name`
Original file line number	Diff line number	Diff line change
`@@ -443,9 +443,9 @@ func setupOptionsVsiExtention(t *testing.T, prefix string, region string, existi`
`443`	`443`	`fmt.Println(err)`
`444`	`444`	`return nil`
`445`	`445`	`}`
`446`		`- // Loop through the vpcs and find the vpc_id when vpc_name is "<prefix>-management-vpc"`
	`446`	`+ // Loop through the vpcs and find the vpc_id when vpc_name is "<prefix>-management"`
`447`	`447`	`for _, vpc := range vpcs {`
`448`		`- if vpc.VpcName == fmt.Sprintf("%s-management-vpc", prefix) {`
	`448`	`+ if vpc.VpcName == fmt.Sprintf("%s-management", prefix) {`
`449`	`449`	`managementVpcID = vpc.VpcID`
`450`	`450`	`}`
`451`	`451`	`}`