Skip to content

Commit b91965c

Browse files
abhay-krishnaabhay-krishna
authored
feat(eks): add L2 constructs to support EKS Hybrid Nodes (#32389)
### Issue # (if applicable) Closes #32362. ### Reason for this change This change allows users to specify the networking primitives for an EKS Hybrid nodes cluster through L2 constructs. ### Description of changes This PR introduces two new top-level fields called `remoteNodeNetworks` and `remotePodNetworks` in the Cluster construct. Together, these allow users to specify the exact CIDRs ranges they want to use for their on-premises nodes and (optionally) pods. The Hybrid nodes feature requires that none of the node and pod CIDRs overlap with each other so I have also added validations for that. ### Description of how you validated changes Added unit tests with different testcases involving different remote network configurations. ### Checklist - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent 1d15d49 commit b91965c

File tree

54 files changed

+10047
-26
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

54 files changed

+10047
-26
lines changed

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.alb-controller-authapi.js.snapshot/integ-eks-stack.template.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -769,7 +769,7 @@
769769
"Arn"
770770
]
771771
},
772-
"AttributesRevision": 3
772+
"AttributesRevision": 4
773773
},
774774
"DependsOn": [
775775
"ClusterCreationRoleDefaultPolicyE8BDFC7B",

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.alb-controller.js.snapshot/aws-cdk-eks-cluster-alb-controller.template.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -767,7 +767,7 @@
767767
"Arn"
768768
]
769769
},
770-
"AttributesRevision": 3
770+
"AttributesRevision": 4
771771
},
772772
"DependsOn": [
773773
"ClusterCreationRoleDefaultPolicyE8BDFC7B",

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-addon.js.snapshot/EksClusterWithAddonStack.template.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -893,7 +893,7 @@
893893
"Arn"
894894
]
895895
},
896-
"AttributesRevision": 3
896+
"AttributesRevision": 4
897897
},
898898
"DependsOn": [
899899
"ClusterCreationRoleDefaultPolicyE8BDFC7B",

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-al2023-nodegroup.js.snapshot/aws-cdk-eks-cluster-al2023-nodegroup-test.template.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -799,7 +799,7 @@
799799
"Arn"
800800
]
801801
},
802-
"AttributesRevision": 3
802+
"AttributesRevision": 4
803803
},
804804
"DependsOn": [
805805
"ClusterCreationRoleDefaultPolicyE8BDFC7B",

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-bottlerocket-ng.js.snapshot/aws-cdk-eks-cluster-bottlerocket-ng-test.template.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -799,7 +799,7 @@
799799
"Arn"
800800
]
801801
},
802-
"AttributesRevision": 3
802+
"AttributesRevision": 4
803803
},
804804
"DependsOn": [
805805
"ClusterCreationRoleDefaultPolicyE8BDFC7B",

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-cluster-handlers-vpc.js.snapshot/aws-cdk-eks-handlers-in-vpc-test.template.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -815,7 +815,7 @@
815815
"Arn"
816816
]
817817
},
818-
"AttributesRevision": 3
818+
"AttributesRevision": 4
819819
},
820820
"DependsOn": [
821821
"EksAllHandlersInVpcStackDefaultVpcIGW916D42F1",

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-cluster-imported.js.snapshot/aws-cdk-eks-import-cluster-test.template.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -841,7 +841,7 @@
841841
"Arn"
842842
]
843843
},
844-
"AttributesRevision": 3
844+
"AttributesRevision": 4
845845
},
846846
"DependsOn": [
847847
"ClusterCreationRoleDefaultPolicyE8BDFC7B",

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-cluster-ipv6.js.snapshot/aws-cdk-eks-cluster-ipv6-test.template.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1236,7 +1236,7 @@
12361236
"Arn"
12371237
]
12381238
},
1239-
"AttributesRevision": 3
1239+
"AttributesRevision": 4
12401240
},
12411241
"DependsOn": [
12421242
"CIDR6",

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-cluster-private-endpoint.js.snapshot/aws-cdk-eks-cluster-private-endpoint-test.template.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -799,7 +799,7 @@
799799
"Arn"
800800
]
801801
},
802-
"AttributesRevision": 3
802+
"AttributesRevision": 4
803803
},
804804
"DependsOn": [
805805
"ClusterCreationRoleDefaultPolicyE8BDFC7B",

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-cluster-tags.js.snapshot/EKSTagStack.template.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -770,7 +770,7 @@
770770
"Arn"
771771
]
772772
},
773-
"AttributesRevision": 3
773+
"AttributesRevision": 4
774774
},
775775
"DependsOn": [
776776
"ClusterCreationRoleDefaultPolicyE8BDFC7B",

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-cluster.js.snapshot/aws-cdk-eks-cluster.template.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1017,7 +1017,7 @@
10171017
"Arn"
10181018
]
10191019
},
1020-
"AttributesRevision": 3
1020+
"AttributesRevision": 4
10211021
},
10221022
"DependsOn": [
10231023
"ClusterCreationRoleDefaultPolicyE8BDFC7B",

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-helm-asset.js.snapshot/aws-cdk-eks-helm-test.template.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -855,7 +855,7 @@
855855
"Arn"
856856
]
857857
},
858-
"AttributesRevision": 3
858+
"AttributesRevision": 4
859859
},
860860
"DependsOn": [
861861
"ClusterCreationRoleDefaultPolicyE8BDFC7B",

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-hybrid-nodes.js.snapshot/asset.0afbe25a9ce0197a83dffd77a86c1560704da246373a4ccafde5476e7e3d0811.zip

31.2 MB
Binary file not shown.

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-hybrid-nodes.js.snapshot/asset.39472b1c2875cf306d4ba429aeccdd34cb49bcf59dbde81f7e6b6cb9deac23a6/cfn-response.js

+106
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎packages/@aws-cdk-testing/framework-integ/test/aws-eks/test/integ.eks-hybrid-nodes.js.snapshot/asset.39472b1c2875cf306d4ba429aeccdd34cb49bcf59dbde81f7e6b6cb9deac23a6/consts.js

+10
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)