Skip to content

Commit b3600de

Browse files
lays147bryantbiggs
andauthored
fix: Add missing Name tag to service security group (#177)
* fix: Add missing Name tag in service sg Signed-off-by: Lays Rodrigues <[email protected]> * chore: Fix up ordering --------- Signed-off-by: Lays Rodrigues <[email protected]> Co-authored-by: Bryant Biggs <[email protected]>
1 parent 62001b5 commit b3600de

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

Diff for: .pre-commit-config.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
repos:
22
- repo: https://github.com/antonbabenko/pre-commit-terraform
3-
rev: v1.83.6
3+
rev: v1.88.0
44
hooks:
55
- id: terraform_fmt
66
- id: terraform_wrapper_module_for_each

Diff for: modules/service/main.tf

+5-1
Original file line numberDiff line numberDiff line change
@@ -1320,7 +1320,11 @@ resource "aws_security_group" "this" {
13201320
description = var.security_group_description
13211321
vpc_id = data.aws_subnet.this[0].vpc_id
13221322

1323-
tags = merge(var.tags, var.security_group_tags)
1323+
tags = merge(
1324+
var.tags,
1325+
{ "Name" = local.security_group_name },
1326+
var.security_group_tags
1327+
)
13241328

13251329
lifecycle {
13261330
create_before_destroy = true

0 commit comments

Comments
 (0)