Skip to content

Commit

Permalink
minor
Browse files Browse the repository at this point in the history
  • Loading branch information
samidbb committed Dec 14, 2023
1 parent 78529d5 commit 7942957
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 8 deletions.
7 changes: 1 addition & 6 deletions modules/security_group/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -35,12 +35,7 @@ resource "aws_security_group" "this_name_prefix" {
vpc_id = var.vpc_id
revoke_rules_on_delete = var.revoke_rules_on_delete

tags = merge(
{
"Name" = format("%s", var.name)
},
var.tags,
)
tags = var.tags

lifecycle {
create_before_destroy = true
Expand Down
4 changes: 2 additions & 2 deletions tests/instance/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,7 @@ locals {
azs = slice(data.aws_availability_zones.available.names, 0, 3)

tags = {
Example = local.name
Repository = "https://github.com/dfds/aws-modules-rds"
"dfds.automation.tool" = "Terraform"
}
}

Expand Down Expand Up @@ -65,6 +64,7 @@ module "rds_instance_test" {
cost_centre = "IT"
data_classification = "public"
enable_default_backup = true
optional_tags = local.tags
}

################################################################################
Expand Down

0 comments on commit 7942957

Please sign in to comment.