Skip to content

Conversation

@fernando-villalba
Copy link
Collaborator

@fernando-villalba fernando-villalba commented Oct 23, 2025

The API spec is now ready to review. You can use this design as reference to check the API definition:

https://github.com/numtide/multigres-operator/blob/multigres-cluster-cr-design/plans/phase-1/final-cr-definition.md

Please note that the pipeline failed due to the KubeBuilder generated controller files, which we could remove if needed, or use as a reference to code forward.

@fernando-villalba fernando-villalba marked this pull request as draft October 23, 2025 11:17
@fernando-villalba fernando-villalba changed the title DRAFT: First new API spec commit, still in draft DRAFT: First new API spec commit Oct 23, 2025
@fernando-villalba fernando-villalba changed the title DRAFT: First new API spec commit New API Spec Oct 24, 2025
@fernando-villalba fernando-villalba marked this pull request as ready for review October 24, 2025 14:54
Copy link
Member

@rytswd rytswd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't go through the entire code yet, but just left a few quick ones for now.

Comment on lines +73 to +88
// StatelessSpec defines the desired state for a scalable, stateless component
// like MultiAdmin, MultiOrch, or MultiGateway.
type StatelessSpec struct {
// Replicas is the desired number of pods.
// +kubebuilder:validation:Minimum=0
// +optional
Replicas *int32 `json:"replicas,omitempty"`

// Affinity defines the pod's scheduling constraints.
// +optional
Affinity *corev1.Affinity `json:"affinity,omitempty"`

// Resources defines the compute resource requirements.
// +optional
corev1.ResourceRequirements `json:"resources,omitempty"`
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because this is used by multiple resources, let's move this to a separate file

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please remove all the code from internal/controller, as we will not be using them (and they are basically useless templates as is).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this was meant to be renamed to "tablegroup_types.go"?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants