Top-level MultiKueue cluster API (kubernetes-sigs#1631)

* [api/v1alpha1] Make MultiKueueCluster an API object.

* [multikueue] Make MultiKueueCluster an API object.

* Fix after rebase

* Review Remarks

* Review Remarks

* Review Remarks

* [API/multikueue] Drop KubeConfig.Name

* Review remarks

* Review remarks

apis/kueue/v1alpha1/multikueueconfig_types.go → apis/kueue/v1alpha1/multikueue_types.go

@@ -22,24 +22,9 @@ import (
 
 const (
 	MultiKueueConfigSecretKey = "kubeconfig"
+	MultiKueueClusterActive   = "Active"
 )
 
-// MultiKueueConfigSpec defines the desired state of MultiKueueConfig
-type MultiKueueConfigSpec struct {
-	// clusters contains the list of configurations for using worker clusters.
-	//
-	// +listType=map
-	// +listMapKey=name
-	// +kubebuilder:validation:MinItems=1
-	// +kubebuilder:validation:MaxItems=100
-	Clusters []MultiKueueCluster `json:"clusters"`
-}
-
-type MultiKueueCluster struct {
-	Name          string        `json:"name"`
-	KubeconfigRef KubeconfigRef `json:"kubeconfigRef"`
-}
-
 type LocationType string
 
 const (
@@ -48,11 +33,11 @@ const (
 	SecretLocationType LocationType = "Secret"
 )
 
-type KubeconfigRef struct {
-	// Name of the cluster inside the given KubeConfig.
-	Name string `json:"name"`
-
+type KubeConfig struct {
 	// Location of the KubeConfig.
+	//
+	// If LocationType is Secret then Location is the name of the secret inside the namespace in
+	// which the kueue controller manager is running. The config should be stored in the "kubeconfig" key.
 	Location string `json:"location"`
 
 	// Type of the KubeConfig location.
@@ -62,21 +47,70 @@ type KubeconfigRef struct {
 	LocationType LocationType `json:"locationType"`
 }
 
-//+genclient
-//+genclient:nonNamespaced
-//+kubebuilder:object:root=true
-//+kubebuilder:storageversion
-//+kubebuilder:resource:scope=Cluster
+type MultiKueueClusterSpec struct {
+	// Information how to connect to the cluster.
+	KubeConfig KubeConfig `json:"kubeConfig"`
+}
+
+type MultiKueueClusterStatus struct {
+	// +optional
+	// +listType=map
+	// +listMapKey=type
+	// +patchStrategy=merge
+	// +patchMergeKey=type
+	Conditions []metav1.Condition `json:"conditions,omitempty" patchStrategy:"merge" patchMergeKey:"type"`
+}
+
+// +genclient
+// +genclient:nonNamespaced
+// +kubebuilder:object:root=true
+// +kubebuilder:storageversion
+// +kubebuilder:subresource:status
+// +kubebuilder:resource:scope=Cluster
+
+// MultiKueueCluster is the Schema for the multikueue API
+type MultiKueueCluster struct {
+	metav1.TypeMeta   `json:",inline"`
+	metav1.ObjectMeta `json:"metadata,omitempty"`
+
+	Spec   MultiKueueClusterSpec   `json:"spec,omitempty"`
+	Status MultiKueueClusterStatus `json:"status,omitempty"`
+}
+
+// +kubebuilder:object:root=true
+
+// MultiKueueClusterList contains a list of MultiKueueCluster
+type MultiKueueClusterList struct {
+	metav1.TypeMeta `json:",inline"`
+	metav1.ListMeta `json:"metadata,omitempty"`
+	Items           []MultiKueueCluster `json:"items"`
+}
+
+// MultiKueueConfigSpec defines the desired state of MultiKueueConfig
+type MultiKueueConfigSpec struct {
+	// List of MultiKueueClusters names where the workloads from the ClusterQueue should be distributed.
+	//
+	// +listType=set
+	// +kubebuilder:validation:MinItems=1
+	// +kubebuilder:validation:MaxItems=10
+	Clusters []string `json:"clusters"`
+}
+
+// +genclient
+// +genclient:nonNamespaced
+// +kubebuilder:object:root=true
+// +kubebuilder:storageversion
+// +kubebuilder:resource:scope=Cluster
 
-// MultiKueueConfig is the Schema for the provisioningrequestconfig API
+// MultiKueueConfig is the Schema for the multikueue API
 type MultiKueueConfig struct {
 	metav1.TypeMeta   `json:",inline"`
 	metav1.ObjectMeta `json:"metadata,omitempty"`
 
 	Spec MultiKueueConfigSpec `json:"spec,omitempty"`
 }
 
-//+kubebuilder:object:root=true
+// +kubebuilder:object:root=true
 
 // MultiKueueConfigList contains a list of MultiKueueConfig
 type MultiKueueConfigList struct {
@@ -86,5 +120,5 @@ type MultiKueueConfigList struct {
 }
 
 func init() {
-	SchemeBuilder.Register(&MultiKueueConfig{}, &MultiKueueConfigList{})
+	SchemeBuilder.Register(&MultiKueueConfig{}, &MultiKueueConfigList{}, &MultiKueueCluster{}, &MultiKueueClusterList{})
 }