Skip to content

Commit b9dfc94

Browse files
fix: need to try/coalesce
1 parent 1081f4f commit b9dfc94

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

autoscaler/autoscaler.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -158,8 +158,8 @@ resource "azurerm_linux_function_app" "autoscaler" {
158158
AUTOSCALING_MAX_KILL = var.autoscaling_configuration.max_terminate != null ? var.autoscaling_configuration.max_terminate : 1
159159
AUTOSCALING_SCALE_DOWN_DELAY = var.autoscaling_configuration.scale_down_delay != null ? var.autoscaling_configuration.scale_down_delay : 0
160160

161-
AZURE_AUTOSCALING_MIN_SIZE = coalesce(var.autoscaling_configuration.scale.min, -1)
162-
AZURE_AUTOSCALING_MAX_SIZE = coalesce(var.autoscaling_configuration.scale.max, 5)
161+
AZURE_AUTOSCALING_MIN_SIZE = coalesce(try(var.autoscaling_configuration.scale.min, null), -1)
162+
AZURE_AUTOSCALING_MAX_SIZE = coalesce(try(var.autoscaling_configuration.scale.max, null), 5)
163163
}
164164

165165
tags = merge(var.tags, {

vmss.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ resource "azurerm_linux_virtual_machine_scale_set" "this" {
122122
location = var.resource_group.location
123123
sku = var.vmss_sku
124124

125-
instances = coalesce(var.autoscaling_configuration.scale.min, var.non_autoscaled_vmss_instances, 2)
125+
instances = coalesce(try(var.autoscaling_configuration.scale.min, null), var.non_autoscaled_vmss_instances, 2)
126126
admin_username = var.admin_username
127127
admin_password = var.admin_password
128128
disable_password_authentication = var.admin_password == null

0 commit comments

Comments
 (0)