Skip to content
This repository was archived by the owner on Jan 16, 2025. It is now read-only.

Commit 448a3a7

Browse files
sgironesnpalm
authored andcommitted
fix(multi-runner): Default value validation error (#2685)
[multi-runner] Default value validation error Avoid validation error when `enable_runner_binaries_syncer=false`. ``` Error: Invalid value for input variable │ │ on .terraform/modules/multi-runner/modules/multi-runner/runners.tf line 13, in module "runners": │ 13: s3_runner_binaries = each.value.runner_config.enable_runner_binaries_syncer ? local.runner_binaries_by_os_and_arch_map["${each.value.runner_config.runner_os}_${each.value.runner_config.runner_architecture}"] : {} │ │ The given value is not suitable for module.multi-runner.module.runners["linux-x64-default"].var.s3_runner_binaries declared at │ .terraform/modules/multi-runner/modules/runners/variables.tf:49,1-30: attributes "arn", "id", and "key" are required. ```
1 parent 08c484c commit 448a3a7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Diff for: modules/multi-runner/runners.tf

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ module "runners" {
1010
"ghr:environment" = "${var.prefix}-${each.key}"
1111
})
1212

13-
s3_runner_binaries = each.value.runner_config.enable_runner_binaries_syncer ? local.runner_binaries_by_os_and_arch_map["${each.value.runner_config.runner_os}_${each.value.runner_config.runner_architecture}"] : {}
13+
s3_runner_binaries = each.value.runner_config.enable_runner_binaries_syncer ? local.runner_binaries_by_os_and_arch_map["${each.value.runner_config.runner_os}_${each.value.runner_config.runner_architecture}"] : null
1414

1515
ssm_paths = {
1616
root = "${local.ssm_root_path}/${var.prefix}-${each.key}"

0 commit comments

Comments
 (0)