diff --git a/aks/application/variables.tf b/aks/application/variables.tf index b76f5f4..ac99e43 100644 --- a/aks/application/variables.tf +++ b/aks/application/variables.tf @@ -37,6 +37,7 @@ variable "cluster_configuration_map" { variable "replicas" { type = number + nullable = false default = 1 description = "Number of application instances" } @@ -64,42 +65,49 @@ variable "docker_image" { variable "command" { type = list(string) + nullable = false default = [] description = "Custom command that overwrites Docker image" } variable "max_memory" { type = string + nullable = false default = "1Gi" description = "Maximum memory of the instance" } variable "is_web" { type = bool + nullable = false default = true description = "Whether this a web application" } variable "web_external_hostnames" { type = list(string) + nullable = false default = [] description = "List of external hostnames for the web application" } variable "web_port" { type = number + nullable = false default = 3000 description = "Port of the web application" } variable "probe_path" { type = string + nullable = false default = "/healthcheck" description = "Path for the liveness and startup probe. The probe can be disabled by setting this to null." } variable "probe_command" { type = list(string) + nullable = false default = [] description = "Command for the liveness and startup probe" } @@ -123,6 +131,7 @@ variable "azure_resource_prefix" { variable "azure_enable_monitoring" { type = bool + nullable = false default = false description = "Whether to enable monitoring of container failures" } diff --git a/aks/application_configuration/variables.tf b/aks/application_configuration/variables.tf index 0979702..e5c415a 100644 --- a/aks/application_configuration/variables.tf +++ b/aks/application_configuration/variables.tf @@ -25,12 +25,14 @@ variable "config_short" { variable "is_rails_application" { type = bool + nullable = false default = false description = "If true, sets config variables for a Rails application" } variable "config_variables" { type = map(string) + nullable = false default = {} description = "Additional configuration variables" } @@ -55,6 +57,7 @@ variable "secret_yaml_key" { variable "secret_variables" { type = map(string) + nullable = false default = {} description = "Additional secret variables" } diff --git a/aks/postgres/variables.tf b/aks/postgres/variables.tf index 3ad5ddd..5364a62 100644 --- a/aks/postgres/variables.tf +++ b/aks/postgres/variables.tf @@ -46,6 +46,7 @@ variable "cluster_configuration_map" { variable "server_docker_image" { type = string + nullable = false default = "postgres:14-alpine" description = "Database image to use with kubernetes deployment, eg. postgis/postgis:14-3.4" } @@ -85,13 +86,15 @@ variable "azure_sku_name" { } variable "azure_enable_high_availability" { - type = bool - default = false + type = bool + nullable = false + default = false } variable "azure_extensions" { - type = list(string) - default = [] + type = list(string) + nullable = false + default = [] } variable "azure_memory_threshold" { @@ -110,12 +113,14 @@ variable "azure_storage_threshold" { } variable "azure_enable_monitoring" { - type = bool - default = true + type = bool + nullable = false + default = true } variable "alert_window_size" { type = string + nullable = false default = "PT5M" description = "The period of time that is used to monitor alert activity e.g PT1M, PT5M, PT15M, PT30M, PT1H, PT6H or PT12H" } @@ -130,6 +135,7 @@ variable "azure_maintenance_window" { } variable "azure_enable_backup_storage" { - type = bool - default = true + type = bool + nullable = false + default = true } diff --git a/aks/redis/variables.tf b/aks/redis/variables.tf index 9f20e91..278b6f9 100644 --- a/aks/redis/variables.tf +++ b/aks/redis/variables.tf @@ -46,6 +46,7 @@ variable "cluster_configuration_map" { variable "server_version" { type = string + nullable = false default = "6" description = "Version of Redis server" } @@ -71,13 +72,15 @@ variable "azure_sku_name" { } variable "azure_minimum_tls_version" { - type = string - default = "1.2" + type = string + nullable = false + default = "1.2" } variable "azure_public_network_access_enabled" { - type = bool - default = false + type = bool + nullable = false + default = false } variable "azure_memory_threshold" { @@ -86,13 +89,15 @@ variable "azure_memory_threshold" { } variable "azure_maxmemory_policy" { - type = string - default = "allkeys-lru" + type = string + nullable = false + default = "allkeys-lru" } variable "azure_enable_monitoring" { - type = bool - default = true + type = bool + nullable = false + default = true } variable "azure_patch_schedule" { @@ -101,12 +106,14 @@ variable "azure_patch_schedule" { start_hour_utc = optional(number), maintenance_window = optional(string) })) - default = [] + nullable = false + default = [] } variable "alert_window_size" { type = string default = "PT5M" + nullable = false description = "The period of time that is used to monitor alert activity e.g PT1M, PT5M, PT15M, PT30M, PT1H, PT6H or PT12H" } diff --git a/domains/environment_domains/variables.tf b/domains/environment_domains/variables.tf index b3da23e..1ae115d 100644 --- a/domains/environment_domains/variables.tf +++ b/domains/environment_domains/variables.tf @@ -13,18 +13,21 @@ variable "host_name" { } variable "null_host_header" { + nullable = false default = false description = "The origin_host_header for the azurerm_cdn_frontdoor_origin resource will be var.host_name (if false) or null (if true). If null then the host name from the incoming request will be used." } variable "rule_set_ids" { - type = list(any) - default = [] + type = list(any) + nullable = false + default = [] } variable "multiple_hosted_zones" { - type = bool - default = false + type = bool + nullable = false + default = false } variable "cached_paths" { @@ -34,11 +37,13 @@ variable "cached_paths" { } variable "exclude_cnames" { + nullable = false default = [] description = "Don't create the CNAME for this record from var.domains. We set this when we want to configure front door for a services domain that we are migrating so we do not need to wait for the certificate to validate and front door to propagate the configuration." } variable "redirect_rules" { + nullable = false default = {} description = <