-
Notifications
You must be signed in to change notification settings - Fork 18
Setup cni scripts with stretch cluster #985
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
base: main
Are you sure you want to change the base?
Conversation
bb8d631 to
d9a1e49
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR introduces infrastructure and configuration for a new CNI (Container Network Interface) prototype scenario in the perf-eval category. The changes enable testing of ipvlan CNI with stretch clusters by providing Terraform configurations for both Azure AKS and AWS EKS deployments, along with shell scripts and ARM templates for setting up dual IP configurations on VMSS nodes.
Key Changes:
- Added Terraform configuration files for deploying CNI prototype test scenarios on Azure and AWS
- Modified Azure network module variables to support multiple address spaces and prefixes (changed from string to list(string))
- Added shell scripts for stretch cluster setup with ipvlan CNI configuration
- Reduced EKS addon timeout defaults from 20m to 5m
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| scenarios/perf-eval/cni-prototype/terraform-test-inputs/azure.json | New test input configuration with run ID and region for Azure deployments |
| scenarios/perf-eval/cni-prototype/terraform-inputs/azure.tfvars | New AKS cluster configuration with dual-stack networking (IPv4/IPv6) and network-plugin set to "none" |
| scenarios/perf-eval/cni-prototype/terraform-inputs/aws.tfvars | New EKS cluster configuration with VPC, subnets, NAT gateways, and node groups for CNI testing |
| modules/terraform/azure/variables.tf | Updated network config variables to accept list of address spaces/prefixes instead of single strings |
| modules/terraform/azure/network/variables.tf | Updated network config variables to accept list of address spaces/prefixes instead of single strings |
| modules/terraform/azure/network/main.tf | Updated to use list-typed address spaces/prefixes directly without wrapping in arrays |
| modules/terraform/aws/eks/main.tf | Reduced EKS addon timeout defaults from 20 minutes to 5 minutes |
| modules/kustomize/stretch/vmss-dual-ipconfig.json | New ARM template for creating VMSS with dual IP configuration for ipvlan networking |
| modules/kustomize/stretch/stretch.sh | New shell script for creating/updating AKS stretch clusters with ipvlan secondary IP configuration |
| modules/kustomize/stretch/setup_ipvlan.sh | New shell script for configuring ipvlan CNI on Azure VM nodes using IMDS metadata |
d9a1e49 to
eb80363
Compare
cfb3c8a to
848c67d
Compare
848c67d to
a39cdab
Compare
No description provided.