refactor: cross-object validation and Terraform Upgrade

README.md

@@ -842,7 +842,7 @@ module "cluster_pattern" {
 
 | Name | Version |
 |------|---------|
-| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.3 |
+| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.9.0 |
 | <a name="requirement_ibm"></a> [ibm](#requirement\_ibm) | >= 1.78.0, < 2.0.0 |
 | <a name="requirement_random"></a> [random](#requirement\_random) | >= 3.4.3, < 4.0.0 |
 | <a name="requirement_time"></a> [time](#requirement\_time) | >= 0.9.1, < 1.0.0 |

dynamic_values/config_modules/appid/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/bastion_vsi_list_to_map/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/cluster_worker_pools/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/clusters/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/cos/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/f5/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/flow_logs/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/get_subnets/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/list_to_map/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/list_to_map_value/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/nested_list_to_map_and_merge/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/service_authorizations/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/vpe/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/vpn/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/config_modules/vsi/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

dynamic_values/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

examples/one-vpc-one-vsi/version.tf

@@ -1,5 +1,5 @@
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
   required_providers {
     ibm = {
       source  = "IBM-Cloud/ibm"

examples/override-example/version.tf

@@ -1,5 +1,5 @@
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
   required_providers {
     ibm = {
       source  = "IBM-Cloud/ibm"

f5_config/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

kms/dynamic_values/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

kms/versions.tf

@@ -3,13 +3,13 @@
 ##############################################################################
 
 terraform {
+  required_version = ">= 1.9.0"
   required_providers {
     ibm = {
       source  = "IBM-Cloud/ibm"
       version = ">= 1.60.0, < 2.0.0"
     }
   }
-  required_version = ">= 1.3"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/acl_rules/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/bastion_gateways/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/bastion_vsi/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/bastion_zone_list/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/cloud_object_storage/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/default_security_group_rules/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/f5_deployments/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/f5_security_group_rules/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/f5_tiers/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/key_management/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/network_acls/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/resource_groups/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/security_groups/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/subnet_cidr/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/vpc/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/vpc_address_prefixes/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/config_modules/vpc_subnet_tiers/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/dynamic_values/versions.tf

@@ -3,7 +3,7 @@
 ##############################################################################
 
 terraform {
-  required_version = ">= 1.3"
+  required_version = ">= 1.9.0"
 }
 
 ##############################################################################

patterns/mixed/config.tf

@@ -45,16 +45,6 @@ locals {
   }
   override_type = var.override_json_string == "" ? "override" : "override_json_string"
 
-  ##############################################################################
-  # VALIDATION FOR SSH_KEY
-  ##############################################################################
-
-  override_validation   = (var.override == false && length(var.override_json_string) == 0) ? true : false
-  sshkey_var_validation = (var.ssh_public_key == null && var.existing_ssh_key_name == null) ? true : false
-
-  # tflint-ignore: terraform_unused_declarations
-  validate_ssh = local.override_validation && local.sshkey_var_validation ? tobool("Invalid input: both ssh_public_key and existing_ssh_key_name variables cannot be null together. Please provide a value for at least one of them.") : true
-
   ##############################################################################
   # Default SSH key
   ##############################################################################

patterns/mixed/variables.tf

@@ -589,6 +589,14 @@ variable "override_json_string" {
   description = "Override default values with a JSON object. Any JSON other than an empty string overrides other configuration changes. You can use the [landing zone configuration tool](https://terraform-ibm-modules.github.io/landing-zone-config-tool/#/home) to create the JSON."
   type        = string
   default     = ""
+
+  validation {
+    condition = !(
+      var.override == false && length(var.override_json_string) == 0 &&
+      var.ssh_public_key == null && var.existing_ssh_key_name == null
+    )
+    error_message = "The ssh_public_key and existing_ssh_key_name cannot both be null when override is false and override_json_string is empty."
+  }
 }
 
 ##############################################################################