fix: Correct capacity_reservation_target within launch templates of…

… both EKS and self managed node groups (terraform-aws-modules#1979)
spring-media · Apr 2, 2022 · 381144e · 381144e
1 parent 6d72456
commit 381144e
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 3 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -1,6 +1,6 @@
 repos:
  - repo: https:/antonbabenko/pre-commit-terraform
- rev: v1.64.0
+ rev: v1.64.1
  hooks:
  - id: terraform_fmt
  - id: terraform_validate

diff --git a/examples/self_managed_node_group/README.md b/examples/self_managed_node_group/README.md
@@ -49,6 +49,7 @@ Note that this example may create resources which cost money. Run `terraform des
 
 | Name | Type |
 |------|------|
+| [aws_ec2_capacity_reservation.targeted](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/ec2_capacity_reservation) | resource |
 | [aws_key_pair.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/key_pair) | resource |
 | [aws_kms_key.ebs](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/kms_key) | resource |
 | [aws_kms_key.eks](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/kms_key) | resource |

diff --git a/examples/self_managed_node_group/main.tf b/examples/self_managed_node_group/main.tf
@@ -202,6 +202,12 @@ module "eks" {
  instance_metadata_tags = "disabled"
  }
 
+ capacity_reservation_specification = {
+ capacity_reservation_target = {
+ capacity_reservation_id = aws_ec2_capacity_reservation.targeted.id
+ }
+ }
+
  create_iam_role = true
  iam_role_name = "self-managed-node-group-complete-example"
  iam_role_use_name_prefix = false
@@ -407,6 +413,14 @@ resource "aws_kms_key" "ebs" {
  policy = data.aws_iam_policy_document.ebs.json
 }
 
+resource "aws_ec2_capacity_reservation" "targeted" {
+ instance_type = "m6i.large"
+ instance_platform = "Linux/UNIX"
+ availability_zone = "${local.region}a"
+ instance_count = 1
+ instance_match_criteria = "targeted"
+}
+
 # This policy is required for the KMS key used for EKS root volumes, so the cluster is allowed to enc/dec/attach encrypted EBS volumes
 data "aws_iam_policy_document" "ebs" {
  # Copy of default KMS policy that lets you manage it

diff --git a/modules/eks-managed-node-group/main.tf b/modules/eks-managed-node-group/main.tf
@@ -94,7 +94,7 @@ resource "aws_launch_template" "this" {
  capacity_reservation_preference = lookup(capacity_reservation_specification.value, "capacity_reservation_preference", null)
 
  dynamic "capacity_reservation_target" {
- for_each = lookup(capacity_reservation_specification.value, "capacity_reservation_target", [])
+ for_each = try([capacity_reservation_specification.value.capacity_reservation_target], [])
  content {
  capacity_reservation_id = lookup(capacity_reservation_target.value, "capacity_reservation_id", null)
  }

diff --git a/modules/self-managed-node-group/main.tf b/modules/self-managed-node-group/main.tf
@@ -97,7 +97,7 @@ resource "aws_launch_template" "this" {
  capacity_reservation_preference = lookup(capacity_reservation_specification.value, "capacity_reservation_preference", null)
 
  dynamic "capacity_reservation_target" {
- for_each = lookup(capacity_reservation_specification.value, "capacity_reservation_target", [])
+ for_each = try([capacity_reservation_specification.value.capacity_reservation_target], [])
  content {
  capacity_reservation_id = lookup(capacity_reservation_target.value, "capacity_reservation_id", null)
  }