Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Aug 19, 2025

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 Copilot AI changed the title [WIP] Revert "feat: Add App Gateway Subnet ID as a field to the Overlay Extension Config CRD" Revert "feat: Add App Gateway Subnet ID as a field to the Overlay Extension Config CRD" Aug 19, 2025
@Copilot Copilot AI requested a review from nairashu August 19, 2025 21:15
Copilot finished work on behalf of nairashu August 19, 2025 21:15
@nairashu nairashu marked this pull request as ready for review August 19, 2025 21:28
@Copilot Copilot AI review requested due to automatic review settings August 19, 2025 21:28
@nairashu nairashu requested a review from a team as a code owner August 19, 2025 21:28
@nairashu nairashu requested a review from BeegiiK August 19, 2025 21:28
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 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.

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