aws / amazon-vpc-cni-plugins

VPC CNI plugins for Amazon ECS and EKS.
Apache License 2.0
68 stars 45 forks source link

Block IMDS access over IPv6 when BlockIMDS is set #84

Closed yinyic closed 1 year ago

yinyic commented 1 year ago

Description of changes:

When the CNI plugins in this repo were implemented, IMDS was accessible only over IPv4. EC2 networking team introduced IMDS access over IPv6 last year. This change extends the BlockIMDS functionality to cover IPV6.

Testing: make unit-test

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

yinyic commented 1 year ago

Change looks good, there is a section that is showing a couple of check failures in the file diff, this is expected, yes? Or is it something new?

That's correct Yash - we didn't make any changes to plugins/vpc-tunnel/e2eTests/e2e_test.go and the tests were broken before this commit.