Add validation for apis

Signed-off-by: Jian Qiu <[email protected]>

Author: qiujian16

addon/v1alpha1/0000_00_addon.open-cluster-management.io_clustermanagementaddons.crd.yaml

@@ -30,6 +30,8 @@ spec:
           ManagedClusterAddOn resource. ClusterManagementAddOn is a cluster-scoped
           resource.
         type: object
+        required:
+        - spec
         properties:
           apiVersion:
             description: 'APIVersion defines the versioned schema of this representation

addon/v1alpha1/types_clustermanagementaddon.go

@@ -23,6 +23,7 @@ type ClusterManagementAddOn struct {
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
 	// spec represents a desired configuration for the agent on the cluster management add-on.
+	// +kubebuilder:validation:Required
 	// +required
 	Spec ClusterManagementAddOnSpec `json:"spec"`
 

cluster/v1/0000_00_clusters.open-cluster-management.io_managedclusters.crd.yaml

@@ -43,6 +43,8 @@ spec:
           on the ManagedCluster pushes the credential to the hub to use against the
           kube-apiserver of the ManagedCluster."
         type: object
+        required:
+        - spec
         properties:
           apiVersion:
             description: 'APIVersion defines the versioned schema of this representation
@@ -60,6 +62,8 @@ spec:
             description: Spec represents a desired configuration for the agent on
               the managed cluster.
             type: object
+            required:
+            - hubAcceptsClient
             properties:
               hubAcceptsClient:
                 description: hubAcceptsClient represents that hub accepts the joining
@@ -80,6 +84,7 @@ spec:
                   60 seconds by default
                 type: integer
                 format: int32
+                default: 60
               managedClusterClientConfigs:
                 description: ManagedClusterClientConfigs represents a list of the
                   apiserver address of the managed cluster. If it is empty, the managed

cluster/v1/types.go

@@ -35,6 +35,8 @@ type ManagedCluster struct {
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
 	// Spec represents a desired configuration for the agent on the managed cluster.
+	// +kubebuilder:validation:Required
+	// +required
 	Spec ManagedClusterSpec `json:"spec"`
 
 	// Status represents the current status of joined managed cluster
@@ -59,11 +61,13 @@ type ManagedClusterSpec struct {
 	// the namespace to grant the permision of access from the agent on the managed cluster.
 	// When the value is set to false, the namespace representing the managed cluster is
 	// deleted.
+	// +kubebuilder:validation:Required
 	// +required
 	HubAcceptsClient bool `json:"hubAcceptsClient"`
 
 	// LeaseDurationSeconds is used to coordinate the lease update time of Klusterlet agents on the managed cluster.
 	// If its value is zero, the Klusterlet agent will update its lease every 60 seconds by default
+	// +kubebuilder:default:=60
 	// +optional
 	LeaseDurationSeconds int32 `json:"leaseDurationSeconds,omitempty"`
 }

cluster/v1alpha1/0000_00_clusters.open-cluster-management.io_managedclustersets.crd.yaml

@@ -28,6 +28,8 @@ spec:
           to update this label, user must have the permission on both the old and
           new ManagedClusterSet."
         type: object
+        required:
+        - spec
         properties:
           apiVersion:
             description: 'APIVersion defines the versioned schema of this representation

cluster/v1alpha1/0000_01_clusters.open-cluster-management.io_managedclustersetbindings.crd.yaml

@@ -22,6 +22,8 @@ spec:
           created in the same namespace can only be distributed to ManagedClusters
           in ManagedClusterSets bound in this namespace by higher level controllers.
         type: object
+        required:
+        - spec
         properties:
           apiVersion:
             description: 'APIVersion defines the versioned schema of this representation
@@ -38,6 +40,8 @@ spec:
           spec:
             description: Spec defines the attributes of ManagedClusterSetBinding.
             type: object
+            required:
+            - clusterSet
             properties:
               clusterSet:
                 description: ClusterSet is the name of the ManagedClusterSet to bind.

cluster/v1alpha1/0000_02_clusters.open-cluster-management.io_clusterclaims.crd.yaml

@@ -21,6 +21,8 @@ spec:
           an identifier that relates the cluster      to the ClusterSet in which it
           belongs.
         type: object
+        required:
+        - spec
         properties:
           apiVersion:
             description: 'APIVersion defines the versioned schema of this representation
@@ -37,6 +39,8 @@ spec:
           spec:
             description: Spec defines the attributes of the ClusterClaim.
             type: object
+            required:
+            - value
             properties:
               value:
                 description: Value is a claim-dependent string

cluster/v1alpha1/0001_02_clusters.open-cluster-management.io_clusterclaims.crd.yaml

@@ -18,6 +18,8 @@ spec:
         a unique identifier for the cluster.   2. clusterset.k8s.io, it contains an
         identifier that relates the cluster      to the ClusterSet in which it belongs.
       type: object
+      required:
+      - spec
       properties:
         apiVersion:
           description: 'APIVersion defines the versioned schema of this representation
@@ -34,6 +36,8 @@ spec:
         spec:
           description: Spec defines the attributes of the ClusterClaim.
           type: object
+          required:
+          - value
           properties:
             value:
               description: Value is a claim-dependent string

cluster/v1alpha1/types.go

@@ -26,6 +26,8 @@ type ManagedClusterSet struct {
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
 	// Spec defines the attributes of the ManagedClusterSet
+	// +kubebuilder:validation:Required
+	// +required
 	Spec ManagedClusterSetSpec `json:"spec"`
 
 	// Status represents the current status of the ManagedClusterSet
@@ -78,6 +80,8 @@ type ManagedClusterSetBinding struct {
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
 	// Spec defines the attributes of ManagedClusterSetBinding.
+	// +kubebuilder:validation:Required
+	// +required
 	Spec ManagedClusterSetBindingSpec `json:"spec"`
 }
 
@@ -88,6 +92,8 @@ type ManagedClusterSetBindingSpec struct {
 	// User is allowed to set this field if they have an RBAC rule to CREATE on the
 	// virtual subresource of managedclustersets/bind.
 	// +kubebuilder:validation:MinLength=1
+	// +kubebuilder:validation:Required
+	// +required
 	ClusterSet string `json:"clusterSet"`
 }
 
@@ -120,14 +126,18 @@ type ClusterClaim struct {
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
 	// Spec defines the attributes of the ClusterClaim.
-	Spec ClusterClaimSpec `json:"spec,omitempty"`
+	// +kubebuilder:validation:Required
+	// +required
+	Spec ClusterClaimSpec `json:"spec"`
 }
 
 type ClusterClaimSpec struct {
 	// Value is a claim-dependent string
 	// +kubebuilder:validation:MaxLength=1024
 	// +kubebuilder:validation:MinLength=1
-	Value string `json:"value,omitempty"`
+	// +kubebuilder:validation:Required
+	// +required
+	Value string `json:"value"`
 }
 
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object

operator/v1/0000_00_operator.open-cluster-management.io_klusterlets.crd.yaml

@@ -20,6 +20,8 @@ spec:
           in the same namespace to allow API requests to the hub for the registration
           protocol.
         type: object
+        required:
+        - spec
         properties:
           apiVersion:
             description: 'APIVersion defines the versioned schema of this representation
@@ -43,6 +45,8 @@ spec:
                   created on hub. The Klusterlet agent generates a random name if
                   it is not set, or discovers the appropriate cluster name on OpenShift.
                 type: string
+                maxLength: 63
+                pattern: ^[a-z0-9]([-a-z0-9]*[a-z0-9])?$
               externalServerURLs:
                 description: ExternalServerURLs represents the a list of apiserver
                   urls and ca bundles that is accessible externally If it is set empty,
@@ -70,6 +74,8 @@ spec:
                   set, the namespace of "open-cluster-management-agent" is used to
                   deploy agent.
                 type: string
+                maxLength: 63
+                pattern: ^[a-z0-9]([-a-z0-9]*[a-z0-9])?$
               registrationImagePullSpec:
                 description: RegistrationImagePullSpec represents the desired image
                   configuration of registration agent.