feat: validate prefixblocklength check to be 0 or 28 based on subnetallocationmode

Author: kmurudi

crd/multitenancy/api/v1alpha1/podnetwork.go

@@ -14,6 +14,8 @@ import (
 // PodNetwork is the Schema for the PodNetworks API
 // +kubebuilder:resource:shortName=pn,scope=Cluster
 // +kubebuilder:subresource:status
+// +kubebuilder:validation:XValidation:rule="!has(self.status.prefixBlockCIDRLength) || self.spec.subnetAllocationMode != 'PrefixBlock' || self.status.prefixBlockCIDRLength == 28",message="prefixBlockCIDRLength must be 28 when subnetAllocationMode is PrefixBlock"
+// +kubebuilder:validation:XValidation:rule="!has(self.status.prefixBlockCIDRLength) || self.spec.subnetAllocationMode != 'Single' || self.status.prefixBlockCIDRLength == 0",message="prefixBlockCIDRLength must be 0 when subnetAllocationMode is Single"
 // +kubebuilder:printcolumn:name="Status",type=string,priority=1,JSONPath=`.status.status`
 // +kubebuilder:printcolumn:name="Address Prefixes",type=string,priority=1,JSONPath=`.status.addressPrefixes`
 // +kubebuilder:printcolumn:name="Network",type=string,priority=1,JSONPath=`.spec.networkID`
@@ -93,7 +95,7 @@ type PodNetworkStatus struct {
 	Status          Status   `json:"status,omitempty"`
 	AddressPrefixes []string `json:"addressPrefixes,omitempty"`
 	// PrefixBlockCIDRLength indicates the prefix length of IP block allocated to the subnet
-	// +kubebuilder:default=28
+	// +kubebuilder:default=0
 	PrefixBlockCIDRLength int `json:"prefixBlockCIDRLength,omitempty"`
 }
 

crd/multitenancy/manifests/multitenancy.acn.azure.com_podnetworks.yaml

@@ -103,7 +103,7 @@ spec:
                   type: string
                 type: array
               prefixBlockCIDRLength:
-                default: 28
+                default: 0
                 description: PrefixBlockCIDRLength indicates the prefix length of
                   IP block allocated to the subnet
                 type: integer
@@ -117,6 +117,13 @@ spec:
                 type: string
             type: object
         type: object
+        x-kubernetes-validations:
+        - message: prefixBlockCIDRLength must be 28 when subnetAllocationMode is PrefixBlock
+          rule: '!has(self.status.prefixBlockCIDRLength) || self.spec.subnetAllocationMode
+            != ''PrefixBlock'' || self.status.prefixBlockCIDRLength == 28'
+        - message: prefixBlockCIDRLength must be 0 when subnetAllocationMode is Single
+          rule: '!has(self.status.prefixBlockCIDRLength) || self.spec.subnetAllocationMode
+            != ''Single'' || self.status.prefixBlockCIDRLength == 0'
     served: true
     storage: true
     subresources: