Skip to content

Conversation

kmurudi
Copy link
Contributor

@kmurudi kmurudi commented Sep 15, 2025

Reason for Change:
Adding new fields that would be required in podnetwork CRD for enabling prefix block IP allocation to pods in swiftv2 flow. Design doc - https://microsoft.sharepoint.com/:w:/t/ContainerNetworking/EbbITCQaW91FrROW1XQ74ewBT8Sp1GfdJ9z5DcuksFH1_g?e=RaqQPT

Requirements:

Notes:

@kmurudi kmurudi requested a review from a team as a code owner September 15, 2025 08:44
@kmurudi kmurudi requested review from Copilot and csfmomo September 15, 2025 08:44
Copy link
Contributor

@Copilot 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 adds new fields to the PodNetwork CRD to support prefix block IP allocation mode for SwiftV2 flow. The changes introduce a subnet allocation mode configuration with two options: Single (default) and PrefixBlock, along with a configurable prefix block size.

  • Added SubnetAllocationMode enum type with Single and PrefixBlock options
  • Added subnetAllocationMode field to PodNetworkSpec with Single as default
  • Added prefixBlockAllocationSize field to PodNetworkStatus with default value of 16

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
crd/multitenancy/manifests/multitenancy.acn.azure.com_podnetworks.yaml Updated CRD manifest with new subnetAllocationMode and prefixBlockAllocationSize fields
crd/multitenancy/api/v1alpha1/podnetwork.go Added SubnetAllocationMode type definition and new fields to PodNetworkSpec and PodNetworkStatus structs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@kmurudi
Copy link
Contributor Author

kmurudi commented Sep 15, 2025

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant