Fix review findings

Author: sbueringer

api/v1beta1/cluster_types.go

@@ -923,7 +923,7 @@ type ClusterNetwork struct {
 type NetworkRanges struct {
 	// cidrBlocks is a list of CIDR blocks.
 	// +kubebuilder:validation:MaxItems=100
-	// +kubebuilder:validation:items:MaxLength=256
+	// +kubebuilder:validation:items:MaxLength=43
 	CIDRBlocks []string `json:"cidrBlocks"`
 }
 

bootstrap/kubeadm/api/v1beta1/kubeadm_types.go

@@ -233,7 +233,7 @@ type ClusterStatus struct {
 type APIEndpoint struct {
 	// advertiseAddress sets the IP address for the API server to advertise.
 	// +optional
-	// +kubebuilder:validation:MaxLength=256
+	// +kubebuilder:validation:MaxLength=39
 	AdvertiseAddress string `json:"advertiseAddress,omitempty"`
 
 	// bindPort sets the secure port for the API Server to bind to.

bootstrap/kubeadm/config/crd/bases/bootstrap.cluster.x-k8s.io_kubeadmconfigs.yaml

@@ -30,13 +30,15 @@ type IPAddressSpec struct {
 	PoolRef corev1.TypedLocalObjectReference `json:"poolRef"`
 
 	// address is the IP address.
+	// +kubebuilder:validation:MaxLength=39
 	Address string `json:"address"`
 
 	// prefix is the prefix of the address.
 	Prefix int `json:"prefix"`
 
 	// gateway is the network gateway of the network the address is from.
 	// +optional
+	// +kubebuilder:validation:MaxLength=39
 	Gateway string `json:"gateway,omitempty"`
 }
 

bootstrap/kubeadm/config/crd/bases/bootstrap.cluster.x-k8s.io_kubeadmconfigtemplates.yaml

@@ -30,15 +30,15 @@ type IPAddressSpec struct {
 	PoolRef corev1.TypedLocalObjectReference `json:"poolRef"`
 
 	// address is the IP address.
-	// +kubebuilder:validation:MaxLength=256
+	// +kubebuilder:validation:MaxLength=39
 	Address string `json:"address"`
 
 	// prefix is the prefix of the address.
 	Prefix int `json:"prefix"`
 
 	// gateway is the network gateway of the network the address is from.
 	// +optional
-	// +kubebuilder:validation:MaxLength=256
+	// +kubebuilder:validation:MaxLength=39
 	Gateway string `json:"gateway,omitempty"`
 }