restructure validation checks to only check for image source type

Signed-off-by: Ankita Thomas <[email protected]>
operator-framework · Oct 1, 2024 · 20dbb55 · 20dbb55
1 parent ce16296
commit 20dbb55
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 10 deletions.
diff --git a/api/core/v1alpha1/clustercatalog_types.go b/api/core/v1alpha1/clustercatalog_types.go
@@ -145,8 +145,7 @@ type ClusterCatalogStatus struct {
 // CatalogSource is a discriminated union of possible sources for a Catalog.
 // CatalogSource contains the sourcing information for a Catalog
 // +union
-// +kubebuilder:validation:XValidation:rule="self.type != 'Image' || has(self.image)",message="source type 'Image' requires image field"
-// +kubebuilder:validation:XValidation:rule="self.type == 'Image' || !has(self.image)",message="image field must only be set for source type 'Image'"
+// +kubebuilder:validation:XValidation:rule="self.type == 'Image' && has(self.image)",message="source type 'Image' requires image field"
 type CatalogSource struct {
  // type is a required reference to the type of source the catalog is sourced from.
  //
@@ -167,8 +166,7 @@ type CatalogSource struct {
 // ResolvedCatalogSource is a discriminated union of resolution information for a Catalog.
 // ResolvedCatalogSource contains the information about a sourced Catalog
 // +union
-// +kubebuilder:validation:XValidation:rule="self.type != 'Image' || has(self.image)",message="source type 'Image' requires image field"
-// +kubebuilder:validation:XValidation:rule="self.type == 'Image' || !has(self.image)",message="image field must only be set for source type 'Image'"
+// +kubebuilder:validation:XValidation:rule="self.type == 'Image' && has(self.image)",message="source type 'Image' requires image field"
 type ResolvedCatalogSource struct {
  // type is a reference to the type of source the catalog is sourced from.
  //

diff --git a/config/base/crd/bases/olm.operatorframework.io_clustercatalogs.yaml b/config/base/crd/bases/olm.operatorframework.io_clustercatalogs.yaml
@@ -119,9 +119,7 @@ spec:
  type: object
  x-kubernetes-validations:
  - message: source type 'Image' requires image field
- rule: self.type != 'Image' || has(self.image)
- - message: image field must only be set for source type 'Image'
- rule: self.type == 'Image' || !has(self.image)
+ rule: self.type == 'Image' && has(self.image)
  required:
  - source
  type: object
@@ -266,9 +264,7 @@ spec:
  type: object
  x-kubernetes-validations:
  - message: source type 'Image' requires image field
- rule: self.type != 'Image' || has(self.image)
- - message: image field must only be set for source type 'Image'
- rule: self.type == 'Image' || !has(self.image)
+ rule: self.type == 'Image' && has(self.image)
  type: object
  required:
  - metadata