Skip to content

Commit

Permalink
Add comments to moved blocks to explain the intent
Browse files Browse the repository at this point in the history
Also reference issue #369 to consider removing
the moved blocks when they are no longer necessary.
  • Loading branch information
jsf9k committed Aug 9, 2024
1 parent e0412e7 commit bd80fc7
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions terraform-post-packer/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,12 @@ data "aws_ami_ids" "historical_amis_x86_64" {
}
}

# This moved block allows us to rename the resources at
# aws_ami_ids.historical_amis to aws_ami_ids.historical_amis_x86_64
# instead of destroying and recreating them with a new name.
#
# TODO: Consider removing this moved block when it is no longer
# needed. Seecisagov/skeleton-packer#369 for more details.
moved {
from = aws_ami_ids.historical_amis
to = aws_ami_ids.historical_amis_x86_64
Expand All @@ -117,6 +123,12 @@ module "ami_launch_permission_x86_64" {
extraorg_account_ids = var.extraorg_account_ids
}

# This moved block allows us to rename the resources at
# module.ami_launch_permission to module.ami_launch_permission_x86_64
# instead of destroying and recreating them with a new name.
#
# TODO: Consider removing this moved block when it is no longer
# needed. Seecisagov/skeleton-packer#369 for more details.
moved {
from = module.ami_launch_permission
to = module.ami_launch_permission_x86_64
Expand Down

0 comments on commit bd80fc7

Please sign in to comment.