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

Adding support for IPv6 worker nodes #773

Merged
merged 12 commits into from
Oct 6, 2021
Merged

Adding support for IPv6 worker nodes #773

merged 12 commits into from
Oct 6, 2021

Conversation

ravisinha0506
Copy link
Contributor

Issue #, if available:

Description of changes

Changes done to add support for IPv6 worker nodes in EKS optimized AMI. Introducing 2 new parameters i.e; ip-family and service-ipv6-cidr. When IpFamily and service-ipv6-cidr are specified, script wouldn't make eks.describe-cluster invocation.

Testing

  • Created both IPv4 and IPv6 worker nodes using the ami with latest commit.

Kubectl get nodes

ip-192-168-2-96.us-west-2.compute.internal    Ready    <none>   4d1h   v1.20.7-eks-135321   192.168.2.96    18.237.128.243   Amazon Linux 2   5.4.144-69.257.amzn2.x86_64   docker://19.3.13

ip-192-168-218-158.us-west-2.compute.internal   Ready    <none>   156m   v1.21.2-eks-55daa9d   2600:1f13:bbe:eb20:bea4:9f79:ac52:7a98   54.187.55.62     Amazon Linux 2   5.4.149-73.259.amzn2.x86_64   docker://20.10.7

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

ravisinha0506 and others added 12 commits July 13, 2021 14:02
merging upstream changes to forked repo
merging with upstream master 07/21
merging with upstream master
changes done to restart docker-daemon on config file update (#718)
pre-fetching sandbox image during bootstrap in containerd runtime (#730)
Adding information about the v20210813 AMI release. (#734)
changing build date and directory for 1.18 and 1.19 (#738)
adjusting permission of pull-sandbox-image.sh
merging with master branch
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.

3 participants