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

ENI tagging #751

Closed
cdenneen opened this issue Apr 24, 2019 · 8 comments
Closed

ENI tagging #751

cdenneen opened this issue Apr 24, 2019 · 8 comments
Labels
area/aws-vpc priority/backlog Not staffed at the moment. Help wanted. stale

Comments

@cdenneen
Copy link

Notice no tagging on any of these ENI's created.
Description for Worker Nodes isn't consistent.
Also not sure why one of the 2 nodes has 2 ENIs and the other has 1.

Screen Shot 2019-04-24 at 12 06 32 PM

@errordeveloper
Copy link
Contributor

Thanks for reporting this @cdenneen! Unfortunately, this is outside of our control. These ENIs are managed by the network driver. I believe that some ENIs also get created by EKS itself, but I'm not fully familiar with those, it's really an implementation detail to me and I would guess that in the future that area may evolve.
The first two ENIs appear like primary ENIs of the nodes that are not used by the network drive, that's why they only have primary IPs and and different description format. The driver uses non-primary ENIs, it creates those and allocates IPs as needed for scheduling pods, but doesn't automatically garbage collect when pods go away.

@errordeveloper
Copy link
Contributor

@christopherhein @tiffanyfay do you think we should open an issue elsewhere? Any other thoughts on this?

@whereisaaron
Copy link

Despite tags for EIP being added to AWS API back in 2017, AWS CloudFormation is yet to support them 😭

@whereisaaron
Copy link

We can track our wait for CF EIP support here:
aws-cloudformation/cloudformation-coverage-roadmap#84

@josegonzalez
Copy link

Seems this was added in November to CF :)

@michaelbeaumont
Copy link
Contributor

ENIs aren't tagged it seems

@martina-if martina-if added priority/blocked priority/backlog Not staffed at the moment. Help wanted. and removed closed/wontfix priority/blocked priority/backlog Not staffed at the moment. Help wanted. labels Sep 14, 2020
@michaelbeaumont michaelbeaumont added priority/backlog Not staffed at the moment. Help wanted. and removed priority/blocked labels Jan 18, 2021
@github-actions
Copy link
Contributor

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the stale label Feb 18, 2021
@github-actions
Copy link
Contributor

This issue was closed because it has been stalled for 5 days with no activity.

torredil pushed a commit to torredil/eksctl that referenced this issue May 20, 2022
Add readiness probe so controller does not report "Ready" prematurely
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/aws-vpc priority/backlog Not staffed at the moment. Help wanted. stale
Projects
None yet
Development

No branches or pull requests

6 participants