fixup! Migrate cluster modules from map(map(string)) to map(object)

Author: pst

aws/_modules/eks/node_pool/variables.tf

@@ -47,6 +47,7 @@ variable "disk_size" {
   type        = number
   default     = 20
   description = "Size in GB of node root volumes."
+  nullable    = false
 }
 
 variable "subnet_ids" {
@@ -72,12 +73,14 @@ variable "tags" {
   type        = map(string)
   description = "AWS tags to set on the node pool. Merged with Kubestack default tags."
   default     = {}
+  nullable    = false
 }
 
 variable "labels" {
   type        = map(string)
   description = "Kubernetes labels to set on the nodes created by the node pool. Merged with Kubestack default labels."
   default     = {}
+  nullable    = false
 }
 
 variable "ami_type" {

aws/_modules/eks/variables.tf

@@ -17,90 +17,105 @@ variable "availability_zones" {
   description = "List of availability_zones for workers."
   type        = list(string)
   default     = []
+  nullable    = false
 }
 
 variable "vpc_cidr" {
   description = "CIDR to use for the VPC."
   type        = string
   default     = "10.0.0.0/16"
+  nullable    = false
 }
 
 variable "vpc_legacy_node_subnets" {
   description = "Whether to keep the legacy node subnets for backwards compatibility."
   type        = bool
   default     = false
+  nullable    = false
 }
 
 variable "vpc_control_subnet_newbits" {
   description = "CIDR newbits to use for the control plane subnets."
   type        = string
   default     = "8"
+  nullable    = false
 }
 
 variable "vpc_dns_hostnames" {
   description = "Enable DNS hostnames in the VPC."
   type        = bool
   default     = false
+  nullable    = false
 }
 
 variable "vpc_dns_support" {
   description = "Enable DNS support in the VPC."
   type        = bool
   default     = true
+  nullable    = false
 }
 
 variable "vpc_node_subnet_newbits" {
   description = "CIDR newbits to use for the node pool subnets."
   type        = string
   default     = "4"
+  nullable    = false
 }
 
 variable "vpc_node_subnet_number_offset" {
   description = "CIDR subnet number offset to use for the node pool subnets."
   type        = string
   default     = "1"
+  nullable    = false
 }
 
 variable "vpc_subnet_map_public_ip" {
   description = "Whether nodes should have public IPs."
   type        = bool
   default     = true
+  nullable    = false
 }
 
 variable "instance_types" {
   description = "Set of AWS instance types to use for nodes."
   type        = set(string)
   default     = []
+  nullable    = false
 }
 
 variable "desired_capacity" {
   description = "Desired number of worker nodes."
   type        = number
   default     = 1
+  nullable    = false
 }
 
 variable "max_size" {
   description = "Maximum number of worker nodes."
   type        = number
   default     = 1
+  nullable    = false
 }
 
 variable "min_size" {
   description = "Minimum number of worker nodes."
   type        = number
   default     = 1
+  nullable    = false
 }
 
 variable "root_device_encrypted" {
   type        = bool
   default     = true
   description = "Whether to encrypt root device volumes of worker nodes."
+  nullable    = false
 }
 
 variable "root_device_volume_size" {
   type        = number
   default     = 20
   description = "Size in GB for root device volumes of worker nodes."
+  nullable    = false
 }
 
 variable "taints" {
@@ -116,42 +131,49 @@ variable "additional_node_tags" {
   type        = map(string)
   default     = {}
   description = "Additional AWS tags to set on the node pool."
+  nullable    = false
 }
 
 variable "aws_auth_map_roles" {
   description = "mapRoles entries added to aws-auth configmap"
   type        = string
   default     = ""
+  nullable    = false
 }
 
 variable "aws_auth_map_users" {
   description = "mapUsers entries added to aws-auth configmap"
   type        = string
   default     = ""
+  nullable    = false
 }
 
 variable "aws_auth_map_accounts" {
   description = "mapAccounts entries added to aws-auth configmap"
   type        = string
   default     = ""
+  nullable    = false
 }
 
 variable "disable_default_ingress" {
   type        = bool
   description = "Whether to disable the default ingress."
   default     = false
+  nullable    = false
 }
 
 variable "enabled_cluster_log_types" {
   type        = list(string)
   description = "List of cluster log types to enable."
   default     = ["api", "audit", "authenticator", "controllerManager", "scheduler"]
+  nullable    = false
 }
 
 variable "disable_openid_connect_provider" {
   type        = bool
   description = "Whether to disable the OpenID connect provider."
   default     = false
+  nullable    = false
 }
 
 variable "cluster_version" {
@@ -163,12 +185,14 @@ variable "cluster_endpoint_private_access" {
   type        = bool
   default     = false
   description = "Whether the Amazon EKS private API server endpoint is enabled (making the kubernetes API endpoint accessible from the internal network)."
+  nullable    = false
 }
 
 variable "cluster_endpoint_public_access" {
   type        = bool
   default     = true
   description = "Whether the Amazon EKS public API server endpoint is enabled (making the kubernetes API endpoint accessible from the public internet)."
+  nullable    = false
 }
 
 variable "cluster_public_access_cidrs" {

azurerm/_modules/aks/variables.tf

@@ -27,60 +27,70 @@ variable "legacy_vnet_name" {
   type        = bool
   description = "Wheter to use the legacy vnet and subnet names."
   default     = false
+  nullable    = false
 }
 
 variable "vnet_address_space" {
   type        = list(string)
   description = "Address space for the virtual network."
   default     = ["10.0.0.0/8"]
+  nullable    = false
 }
 
 variable "subnet_address_prefixes" {
   type        = list(string)
   description = "Address space for the cluster subnet."
   default     = ["10.1.0.0/16"]
+  nullable    = false
 }
 
 variable "subnet_service_endpoints" {
   type        = list(string)
   description = "List of service endpoints to expose on the subnet."
   default     = []
+  nullable    = false
 }
 
 variable "network_plugin" {
   type        = string
   description = "Network plugin to use. Set to 'azure' for CNI; 'kubenet' for Basic"
   default     = "kubenet"
+  nullable    = false
 }
 
 variable "network_policy" {
   type        = string
   description = "Network policy to use. Set to 'azure' for CNI; 'calico' for Basic"
   default     = "calico"
+  nullable    = false
 }
 
 variable "dns_prefix" {
   type        = string
   description = "DNS prefix of AKS cluster API server."
   default     = "api"
+  nullable    = false
 }
 
 variable "service_cidr" {
   type        = string
   description = "CIDR range for kubernetes service."
   default     = "10.0.0.0/16"
+  nullable    = false
 }
 
 variable "dns_service_ip" {
   type        = string
   description = "IP address for the kube-dns service. Must be within service_cidr."
   default     = "10.0.0.10"
+  nullable    = false
 }
 
 variable "pod_cidr" {
   type        = string
   description = "CIDR range for pods."
   default     = "10.244.0.0/16"
+  nullable    = false
 }
 
 variable "max_pods" {
@@ -102,30 +112,35 @@ variable "default_node_pool" {
   })
   description = "Default node pool configuration."
   default     = {}
+  nullable    = false
 }
 
 variable "disable_default_ingress" {
   type        = bool
   description = "Whether to disable the default ingress."
   default     = false
+  nullable    = false
 }
 
 variable "default_ingress_ip_zones" {
   type        = list(string)
   description = "Explicit list of zones for the ingress public IP."
   default     = []
+  nullable    = false
 }
 
 variable "enable_azure_policy_agent" {
   type        = bool
   description = "whether to deploy the Azure policy agent to the cluster"
   default     = false
+  nullable    = false
 }
 
 variable "disable_managed_identities" {
   type        = bool
   description = "Keep using legacy service principal instead of new managed identities."
   default     = false
+  nullable    = false
 }
 
 variable "user_assigned_identity_id" {
@@ -137,6 +152,7 @@ variable "sku_tier" {
   type        = string
   description = "The SKU tier to use for the cluster. Possible values are 'Free' and 'Paid'. Defaults to 'Free'."
   default     = "Free"
+  nullable    = false
 }
 
 variable "kubernetes_version" {
@@ -153,25 +169,29 @@ variable "enable_log_analytics" {
   type        = bool
   description = "whether to deploy the Azure Log analytics with the cluster"
   default     = true
+  nullable    = false
 }
 
 variable "availability_zones" {
   type        = list(string)
   description = "The list of availability zones to create the node pool in"
   default     = []
+  nullable    = false
 }
 
 
 variable "keda_enabled" {
   type        = bool
   description = "Whether KEDA Autoscaler should be enabled for the cluster."
   default     = false
+  nullable    = false
 }
 
 variable "vertical_pod_autoscaler_enabled" {
   type        = bool
   description = "Whether Vertical Pod Autoscaler should be enabled for the cluster."
   default     = false
+  nullable    = false
 }
 
 variable "upgrade_settings" {
@@ -182,4 +202,5 @@ variable "upgrade_settings" {
   })
   description = "Upgrade settings for the default node pool."
   default     = {}
+  nullable    = false
 }