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

VPC Endpoints should add Name tag by default #1150

Closed
aldenquimby opened this issue Dec 17, 2024 · 2 comments
Closed

VPC Endpoints should add Name tag by default #1150

aldenquimby opened this issue Dec 17, 2024 · 2 comments

Comments

@aldenquimby
Copy link

Is your request related to a new offering from AWS?

No

Is your request related to a problem? Please describe.

When adding vpc-endpoints, I currently add a tag for Name on each, like this:

    execute-api = {
      service = "execute-api"
      tags    = { Name = "execute-api-vpc-endpoint" }
    }
    textract-fips = {
      service = "textract-fips"
      tags    = { Name = "textract-fips-vpc-endpoint" }
    }
    transcribe = {
      service = "transcribe"
      tags    = { Name = "transcribe-vpc-endpoint" }
    }

Describe the solution you'd like.

It would be nice if Name tags were auto-added if service is specified, so I could do this instead:

    execute-api = {
      service = "execute-api"
    }
    textract-fips = {
      service = "textract-fips"
    }
    transcribe = {
      service = "transcribe"
    }

Describe alternatives you've considered.

Manually specifying Names works, it's just verbose

Additional context

  • The security group within the vpc-endpoints module already auto-adds a Name tag, which is nice:
    tags = merge(
    var.tags,
    var.security_group_tags,
    { "Name" = try(coalesce(var.security_group_name, var.security_group_name_prefix), "") },
    )
@sebastianczech
Copy link
Contributor

@aldenquimby I prepared PR #1151 , which I hope resolves your problem

@aldenquimby
Copy link
Author

@sebastianczech excellent! Thank you. Exactly what I was hoping for

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants