Validate memcached replicas to be >= 1

openstack-k8s-operators · Jul 30, 2024 · 0df5f68 · 0df5f68
1 parent 84df6df
commit 0df5f68
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/apis/bases/memcached.openstack.org_memcacheds.yaml b/apis/bases/memcached.openstack.org_memcacheds.yaml
@@ -52,6 +52,8 @@ spec:
                 default: 1
                 description: Size of the memcached cluster
                 format: int32
+                maximum: 32
+                minimum: 1
                 type: integer
               tls:
                 description: TLS settings for memcached service
@@ -66,6 +68,7 @@ spec:
                 type: object
             required:
             - containerImage
+            - replicas
             type: object
           status:
             description: MemcachedStatus defines the observed state of Memcached

diff --git a/apis/memcached/v1beta1/memcached_types.go b/apis/memcached/v1beta1/memcached_types.go
@@ -46,8 +46,9 @@ type MemcachedSpec struct {
 // MemcachedSpecCore - this version is used by the OpenStackControlplane CR (no container images)
 type MemcachedSpecCore struct {
 
-	// +kubebuilder:validation:Optional
 	// +kubebuilder:default=1
+	// +kubebuilder:validation:Minimum=1
+	// +kubebuilder:validation:Maximum=32
 	// Size of the memcached cluster
 	Replicas *int32 `json:"replicas"`
 

diff --git a/config/crd/bases/memcached.openstack.org_memcacheds.yaml b/config/crd/bases/memcached.openstack.org_memcacheds.yaml
@@ -52,6 +52,8 @@ spec:
                 default: 1
                 description: Size of the memcached cluster
                 format: int32
+                maximum: 32
+                minimum: 1
                 type: integer
               tls:
                 description: TLS settings for memcached service
@@ -66,6 +68,7 @@ spec:
                 type: object
             required:
             - containerImage
+            - replicas
             type: object
           status:
             description: MemcachedStatus defines the observed state of Memcached