Skip to content

Commit

Permalink
Fix launch template market option expansion (terraform-aws-modules#508)
Browse files Browse the repository at this point in the history
* Fix launch template market option expansion

* Update changelog
  • Loading branch information
to266 authored and max-rocket-internet committed Sep 16, 2019
1 parent f88220a commit 4f4d9c3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,9 @@ project adheres to [Semantic Versioning](http://semver.org/).
- Fixed errors sometimes happening during destroy due to usage of coalesce() in local.tf (by @petrikero)
- Removed historical mention of adding caller's IPv4 to cluster security group (by @dpiddockcmp)
- Wrapped `kubelet_extra_args` in double quotes instead of singe quotes (by @nxf5025)
- Write your awesome change here (by @you)
- Make terraform plan more consistent and avoid unnecessary "(known after apply)" (by @barryib)
- Made sure that `market_type` was correctly passed to `workers_launch_template` (by @to266)
- Write your awesome change here (by @you)

# History

Expand Down
3 changes: 1 addition & 2 deletions workers_launch_template.tf
Original file line number Diff line number Diff line change
Expand Up @@ -316,10 +316,9 @@ resource "aws_launch_template" "workers_launch_template" {
}

dynamic instance_market_options {
iterator = item
for_each = lookup(var.worker_groups_launch_template[count.index], "market_type", null) == null ? [] : list(lookup(var.worker_groups_launch_template[count.index], "market_type", null))
content {
market_type = item.value
market_type = instance_market_options.value
}
}

Expand Down

0 comments on commit 4f4d9c3

Please sign in to comment.