Revert "feat: Add App Gateway Subnet ID as a field to the Overlay Extension Config CRD"

[Copilot]

This PR reverts the changes made in PR #3925 which added the App Gateway Subnet ID field to the OverlayExtensionConfig CRD.

Changes Reverted

The following field has been removed from the OverlayExtensionConfigSpec:

// AppGatewaySubnetGUID field defines a subnet delegated to Application Gateway that should be able to reach routing domain ip addresses. This needs to be a valid GUID.
// +kubebuilder:validation:Optional
// +kubebuilder:validation:XValidation:rule="self == oldSelf",message="Value is immutable"
AppGatewaySubnetGUID string `json:"appGatewaySubnetGUID,omitempty"`

Files Modified

1. crd/overlayextensionconfig/api/v1alpha1/overlayextensionconfig_types.go

   • Removed the AppGatewaySubnetGUID field and its associated kubebuilder validation annotations

2. crd/overlayextensionconfig/manifests/acn.azure.com_overlayextensionconfigs.yaml

   • Regenerated CRD manifest with the field removed from the OpenAPI schema

Testing

• All existing tests continue to pass
• CRD package builds successfully
• No references to the removed field remain in the codebase
• Generated manifests are consistent with the Go struct definition

The OverlayExtensionConfig CRD is now restored to its state before PR #3925, containing only the original ExtensionIPRange field in the spec.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[Copilot]

Pull Request Overview

This PR reverts changes from PR #3925 that added an App Gateway Subnet ID field to the OverlayExtensionConfig Custom Resource Definition (CRD). The revert removes the AppGatewaySubnetGUID field and its associated validation annotations.

• Removes the AppGatewaySubnetGUID field from the OverlayExtensionConfigSpec struct
• Eliminates kubebuilder validation annotations for the removed field
• Regenerates the CRD manifest to reflect the structural changes

Reviewed Changes

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

File	Description
crd/overlayextensionconfig/api/v1alpha1/overlayextensionconfig_types.go	Removes AppGatewaySubnetGUID field and validation annotations from Go struct
crd/overlayextensionconfig/manifests/acn.azure.com_overlayextensionconfigs.yaml	Updates generated CRD manifest to remove appGatewaySubnetGUID property from OpenAPI schema

---

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

[nairashu]

/azp run Azure Container Networking PR

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[github-actions]

This pull request is stale because it has been open for 2 weeks with no activity. Remove stale label or comment or this will be closed in 7 days