api: refactor MultiAdmin config to fix recursive nesting

Removes the `ComponentConfig` struct to prevent `CoreTemplate` from
referencing other templates and to simplify the API structure.

Changes:
- Replaces `ComponentConfig` with a specialized `MultiAdminConfig` in `MultigresCluster`, enforcing "Spec OR TemplateRef" validation.
- Updates `CoreTemplate` to use `StatelessSpec` directly for `MultiAdmin`, flattening the YAML structure (removing the `spec` nesting) and preventing recursive template references.
- Deletes the unused `ComponentConfig` from `common_types.go`.

Author: fernando-villalba

api/v1alpha1/common_types.go

@@ -56,22 +56,6 @@ type StatelessSpec struct {
 	PodLabels map[string]string `json:"podLabels,omitempty"`
 }
 
-// ComponentConfig is a generic wrapper for components that support
-// inline spec OR template reference.
-// +kubebuilder:validation:XValidation:rule="has(self.spec) || has(self.templateRef)",message="must specify either 'spec' or 'templateRef'"
-// +kubebuilder:validation:XValidation:rule="!(has(self.spec) && has(self.templateRef))",message="cannot specify both 'spec' and 'templateRef'"
-type ComponentConfig struct {
-	// Spec defines the inline configuration.
-	// +optional
-	Spec *StatelessSpec `json:"spec,omitempty"`
-
-	// TemplateRef refers to a CoreTemplate to load configuration from.
-	// +optional
-	// +kubebuilder:validation:MinLength=1
-	// +kubebuilder:validation:MaxLength=63
-	TemplateRef string `json:"templateRef,omitempty"`
-}
-
 // StorageSpec defines the storage configuration.
 type StorageSpec struct {
 	// Size of the persistent volume.

api/v1alpha1/coretemplate_types.go

@@ -32,7 +32,7 @@ type CoreTemplateSpec struct {
 
 	// MultiAdmin configuration.
 	// +optional
-	MultiAdmin *ComponentConfig `json:"multiadmin,omitempty"`
+	MultiAdmin *StatelessSpec `json:"multiadmin,omitempty"`
 }
 
 // ============================================================================

api/v1alpha1/multigrescluster_types.go

@@ -42,7 +42,7 @@ type MultigresClusterSpec struct {
 
 	// MultiAdmin defines the configuration for the MultiAdmin component.
 	// +optional
-	MultiAdmin ComponentConfig `json:"multiadmin,omitempty"`
+	MultiAdmin MultiAdminConfig `json:"multiadmin,omitempty"`
 
 	// Cells defines the list of cells (failure domains) in the cluster.
 	// +optional
@@ -121,6 +121,26 @@ type TemplateDefaults struct {
 	ShardTemplate string `json:"shardTemplate,omitempty"`
 }
 
+// ============================================================================
+// MultiAdmin Config Section Specs
+// ============================================================================
+
+// MultiAdminConfig defines the configuration for MultiAdmin in the Cluster.
+// It allows either an inline spec OR a reference to a CoreTemplate.
+// +kubebuilder:validation:XValidation:rule="has(self.spec) || has(self.templateRef)",message="must specify either 'spec' or 'templateRef'"
+// +kubebuilder:validation:XValidation:rule="!(has(self.spec) && has(self.templateRef))",message="cannot specify both 'spec' and 'templateRef'"
+type MultiAdminConfig struct {
+	// Spec defines the inline configuration.
+	// +optional
+	Spec *StatelessSpec `json:"spec,omitempty"`
+
+	// TemplateRef refers to a CoreTemplate to load configuration from.
+	// +optional
+	// +kubebuilder:validation:MinLength=1
+	// +kubebuilder:validation:MaxLength=63
+	TemplateRef string `json:"templateRef,omitempty"`
+}
+
 // ============================================================================
 // Cell Config Section Specs
 // ============================================================================