Skip to content
This repository was archived by the owner on Jan 25, 2026. It is now read-only.

misc: Align resource naming and tags#15

Merged
jongwooo merged 1 commit intomainfrom
misc/align-resource-naming-and-tags
Jan 7, 2026
Merged

misc: Align resource naming and tags#15
jongwooo merged 1 commit intomainfrom
misc/align-resource-naming-and-tags

Conversation

@jongwooo
Copy link
Contributor

@jongwooo jongwooo commented Jan 7, 2026

This pull request introduces minor improvements to the AWS Terraform configuration, focusing on resource naming consistency and better resource tagging.

Resource naming and tagging updates:

  • Renamed the aws_ecs_cluster_capacity_providers resource from capacity_provider_association to capacity_provider_assoc in Modules/AWS/ECS/main.tf for consistency with naming conventions.
  • Added a Name tag to the aws_security_group.sg_db_access_instance resource in Modules/AWS/SecurityGroup/main.tf to improve resource identification.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces minor improvements to AWS Terraform configuration by standardizing resource naming and ensuring consistent tagging across security groups.

  • Shortened the aws_ecs_cluster_capacity_providers resource name for consistency with naming conventions
  • Added missing Name tag to the database access security group, aligning it with other security group resources

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
Modules/AWS/ECS/main.tf Renamed capacity_provider_association to capacity_provider_assoc for naming consistency
Modules/AWS/SecurityGroup/main.tf Added Name tag to sg_db_access_instance resource, matching the tagging pattern of other security groups

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jongwooo jongwooo merged commit 6f6b08a into main Jan 7, 2026
7 checks passed
@jongwooo jongwooo deleted the misc/align-resource-naming-and-tags branch January 7, 2026 15:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant