Skip to content

Commit

Permalink
fix: Outputs for pool need to account for complexity (philips-labs#1970)
Browse files Browse the repository at this point in the history
* fix: outputs for pool need to account for complexity

As the pool is optional the output needs to take this into account and access the first instance in the list. We also must output null if not set.

* Ensure pool role is output from main module
  • Loading branch information
ScottGuymer authored Apr 25, 2022
1 parent cc992ae commit 2d92906
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion modules/runners/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -23,5 +23,5 @@ output "role_scale_down" {
}

output "role_pool" {
value = length(var.pool_config) == 0 ? {} : module.pool.role_pool
value = length(var.pool_config) == 0 ? null : module.pool[0].role_pool
}
1 change: 1 addition & 0 deletions outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ output "runners" {
role_runner = module.runners.role_runner
role_scale_up = module.runners.role_scale_up
role_scale_down = module.runners.role_scale_down
role_pool = module.runners.role_pool
}
}

Expand Down

0 comments on commit 2d92906

Please sign in to comment.