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

tag the eip for the nat gateway #34

Merged
merged 2 commits into from
Feb 17, 2021
Merged

tag the eip for the nat gateway #34

merged 2 commits into from
Feb 17, 2021

Conversation

ekristen
Copy link
Contributor

what

  • apply the same tags to the aws_eip for the nat_gateway as the other resources

why

  • naming and tagging consistency
  • to be able to know that the EIP allocated was for the NAT gateway

@ekristen ekristen requested review from a team as code owners February 16, 2021 23:47
@ekristen ekristen requested a review from a team as a code owner February 16, 2021 23:48
@ekristen ekristen requested review from jhosteny and 3h4x and removed request for a team February 16, 2021 23:48
@Gowiem
Copy link
Member

Gowiem commented Feb 17, 2021

/test all

@Gowiem Gowiem merged commit 9e4585e into cloudposse:master Feb 17, 2021
Nuru added a commit that referenced this pull request Apr 18, 2021
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

Successfully merging this pull request may close these issues.

None yet

3 participants