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

Update creating-a-vpc.md #780

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

bareeqakhan
Copy link

Removed 12th point from 'Public and Private subnets' section since 'AssignIpv6AddressOnCreation' is supported in public subnets in CFN.

Issue #, if available:
In the CFN template provided for VPC with public and private subnets (IPv4 and IPv6), e.g. https://s3.us-west-2.amazonaws.com/amazon-eks/cloudformation/2020-10-29/amazon-eks-ipv6-vpc-public-private-subnets.yaml a limitation has been mentioned -
"Note: AssignIpv6AddressOnCreation: true is required but a CFN limitation right now, need to add this manually"
Due to which we need to manually enable the auto-assign IPv6 address option for the public subnets that were created by the template (as per point 12)
Since this limitation has already been addressed by CFN, so 'AssignIpv6AddressOnCreation' property is supported in public subnets as well and hence manually performing this step is not required now

Description of changes:

  • Please update the CFN template to include 'AssignIpv6AddressOnCreation: true' in public subnets as well {See attachment for updated template EKS-IPv4-IPv6.txt }
  • Remove point 12 that describes how to manually enable the auto-assign IPv6 address option for the public subnets

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

Removed 12th point from 'Public and Private subnets' section since 'AssignIpv6AddressOnCreation' is supported in public subnets in CFN as well.
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.

2 participants