-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
feat: Allow tagging on per-subnet basis #901
base: master
Are you sure you want to change the base?
feat: Allow tagging on per-subnet basis #901
Conversation
9f2a4d3
to
d13247b
Compare
This is now live in our infrastructure. I also added the ability to customize route table tagging. |
d13247b
to
62114c4
Compare
I've completed the rollout of the new tagging system throughout our infrastructure, using the code from this pull request in production. No problems uncovered. |
Awesome! |
I think this behavior is what we are looking for. We wanted to be able to have 2 1 for our EKS cluster's worker nodes, and 1 for some other random servers that still survive post EKS migration. So we could just target the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh, I see now that tags per az is no use when you have more than one private subnet per az
I'd love to see this merged, this would address our use case as well: We'll need individual tags per sub-net, e.g. to control the assignment of load balancers to specific sub-nets in an EKS scenario, which is controlled by sub-net tags. |
Once this Pull Request is merged, I will use it right away |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have been using this branch in production for 3 weeks without any issues.
@antonbabenko would you mind taking a look at this PR please, it's a really helpful and thorough addition, thanks |
This PR has been automatically marked as stale because it has been open 30 days |
keep |
Any chance you can fix the conflicts, id really love to get this one merged in. |
There's not much point in fixing merge conflicts until we get confirmation from a maintainer that they are interested in merging the PR. |
This PR has been automatically marked as stale because it has been open 30 days |
i need this PR. |
We're hitting the same needs for our EKS setup at the moment: need to tag secondary subnets differently to tell karpenter to slowly start moving the nodes towards these subnets. What are the plans for moving this PR forward? |
I'm running into some of the same issues in our environment as well, needing to tag specific subnets differently due to different uses. |
Hi, hitting the same needs here, it would be great to have this PR to move forward ;) |
I think so! |
This PR has been automatically marked as stale because it has been open 30 days |
Keep |
Added the ability to customize NAT EIP and NAT Gateway tags in the same way as the other resources |
@antonbabenko @bryantbiggs Any chance you could review this PR? This is a much requested feature. Thanks! |
This PR has been automatically marked as stale because it has been open 30 days |
keep |
This PR has been automatically marked as stale because it has been open 30 days |
Not stale. @bryantbiggs would you be able to disable stale-bot on this PR to save people some time? |
Hi, hitting the same needs here, please move this PR forward!!!! |
tf-controller plan output: No changes. Your infrastructure matches the configuration.
Terraform has compared your real infrastructure against your configuration
and found no differences, so no changes are needed.
To apply this plan, please merge this pull request. |
@bryantbiggs what needs to be done here to get this PR merged? If the conflicts are resolved will that be enough? |
Hello team, any updates on when this PR will be merged? |
Description
New variables
public_subnet_tags_per_subnet
,private_subnet_tags_per_subnet
, etc. Optional, no effect if not specified. If specified, they are additional tags to apply to each respective public and private subnet.Extend the ability to allow naming and tagging route tables, EIPs, and NAT Gateways on per-subnet basis as well, add variables for that.
Motivation and Context
Previously, it was only possible to customize arbitrary tags on a per-AZ basis, not a per-subnet basis. You could customize the
Name
tag on a per-subnet basis but not any other tag.The VPCs in our environment have several different types of subnets. We would like to tag them differently, e.g. for cost attribution and reference in external Terraform code. Currently this is not possible: subnets can only be disambiguated by VPC, AZ, and visibility.
Breaking Changes
No breaking changes
How Has This Been Tested?
examples/*
to demonstrate and validate my change(s)examples/*
projectspre-commit run -a
on my pull requestThis is currently running in production