Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add tags to aws_appautoscaling_target #65

Merged
merged 1 commit into from
Aug 6, 2024
Merged

feat: Add tags to aws_appautoscaling_target #65

merged 1 commit into from
Aug 6, 2024

Conversation

MaxymVlasov
Copy link
Contributor

what

Add missed tags

why

Tags were introduced for all aws_appautoscaling_target resources created after 2023-03-20.

That blocks us from enforcing policy checks for missing tags.

references

https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/appautoscaling_target

Also, after release a new version here, we need release in https://github.com/cloudposse/terraform-aws-dynamodb/blob/a0cfac2e8c2584125596089a3185fc4a057410e4/main.tf#L153-L155

Tags introduced for all `aws_appautoscaling_target` resources created after 2023-03-20
@MaxymVlasov MaxymVlasov requested review from a team as code owners August 6, 2024 12:05
@mergify mergify bot added the triage Needs triage label Aug 6, 2024
@aknysh
Copy link
Member

aknysh commented Aug 6, 2024

/terratest

@aknysh aknysh added minor New features that do not break anything and removed triage Needs triage labels Aug 6, 2024
Copy link
Member

@aknysh aknysh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @MaxymVlasov

@aknysh aknysh merged commit e61cbb5 into cloudposse:main Aug 6, 2024
45 checks passed
@MaxymVlasov MaxymVlasov deleted the patch-1 branch August 6, 2024 14:39
Copy link

github-actions bot commented Aug 6, 2024

These changes were released in v0.1.0.

MaxymVlasov added a commit to MaxymVlasov/terraform-aws-dynamodb that referenced this pull request Aug 6, 2024
Update `terraform-aws-dynamodb-autoscaler` module to set tags for `aws_appautoscaling_target` resources

Related PR: 
cloudposse/terraform-aws-dynamodb-autoscaler#65
aknysh pushed a commit that referenced this pull request Aug 6, 2024
aknysh pushed a commit to cloudposse/terraform-aws-dynamodb that referenced this pull request Aug 6, 2024
* feat: Add tags to aws_appautoscaling_target

Update `terraform-aws-dynamodb-autoscaler` module to set tags for `aws_appautoscaling_target` resources

Related PR: 
cloudposse/terraform-aws-dynamodb-autoscaler#65

* Update main.tf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor New features that do not break anything
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants