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

Setting MTU on ipv4 egress on a ipv6 only cluster #2949

Closed
umutuzgur opened this issue Jun 10, 2024 · 4 comments · Fixed by #2951
Closed

Setting MTU on ipv4 egress on a ipv6 only cluster #2949

umutuzgur opened this issue Jun 10, 2024 · 4 comments · Fixed by #2951
Labels

Comments

@umutuzgur
Copy link

What happened:

We are trying to change the MTU values on all interfaces on our ipv6 only cluster with ipv4 egress. When we set the AWS_VPC_ENI_MTU value, this only affects the eth0 interface but not the v4if0 interface where ipv4 egress is supported. Which values do we need to set so the MTU values are persisted on both interfaces?

Environment:

  • Kubernetes version (use kubectl version): v1.30.0-eks-036c24b
  • CNI Version: v1.18.1-eksbuild.3
  • OS (e.g: cat /etc/os-release): Amazon Linux 2
  • Kernel (e.g. uname -a): 5.10.213-201.855.amzn2.x86_64
@umutuzgur
Copy link
Author

This might become a feature request or a bug depending on how it is interpreted but we found the culprit to be this here https://github.com/aws/amazon-vpc-cni-k8s/blob/master/misc/10-aws.conflist#L18. When that line is turned to __MTU__, everything works as expected

@orsenthil
Copy link
Member

This may be a bug. The environment variable AWS_VPC_ENI_MTU is picked up and assigned to MTU
Since the value was set to 9001, I assume this wasn't getting set for egress-cni property for IPV4 Egress on IPV6 Cluster.

@benben
Copy link
Contributor

benben commented Jun 11, 2024

@orsenthil thanks for accepting this as a bug. I sent the change on how we fixed it with more context in #2951

Please let me know in case you need anything else. Thanks!

Copy link

github-actions bot commented Jul 1, 2024

This issue is now closed. Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.

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

Successfully merging a pull request may close this issue.

3 participants