Skip to content

Conversation

@alyssa1303
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings December 19, 2025 22:20
Copy link
Contributor

Copilot AI left a 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

@alyssa1303 alyssa1303 force-pushed the setup-cni branch 3 times, most recently from cfb3c8a to 848c67d Compare December 20, 2025 01:22
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