[Feature] Add spec validation for MLExtension (#1807)

Author: ajanikow

CHANGELOG.md

@@ -54,6 +54,7 @@
 - (Bugfix) Fix ArangoRoute Target switch in case of temporary error
 - (Bugfix) Fix IntOrString Schema Type
 - (Feature) Enable Operator Handler StackTraces
+- (Feature) Add spec validation for MLExtension
 
 ## [1.2.43](https://github.com/arangodb/kube-arangodb/tree/1.2.43) (2024-10-14)
 - (Feature) ArangoRoute CRD

pkg/apis/ml/v1beta1/extension_spec.go

@@ -112,10 +112,17 @@ func (a *ArangoMLExtensionSpec) Validate() error {
 
 	return shared.WithErrors(shared.PrefixResourceErrors("spec",
 		shared.PrefixResourceErrors("metadataService", a.GetMetadataService().Validate()),
-		shared.PrefixResourceErrors("storage", shared.ValidateOptionalInterface(a.GetStorage())),
+		shared.PrefixResourceErrors("storage", func() error {
+			if a.GetStorageType() == ArangoMLExtensionSpecStorageTypeExtension {
+				return shared.ValidateRequiredInterface(a.Storage)
+			}
+
+			return nil
+		}()),
 		shared.PrefixResourceErrors("init", a.GetInit().Validate()),
 		shared.PrefixResourceErrors("deployment", a.GetDeployment().Validate()),
 		shared.PrefixResourceErrors("jobsTemplates", a.GetJobsTemplates().Validate()),
 		shared.PrefixResourceErrors("integrationSidecar", a.GetIntegrationSidecar().Validate()),
+		shared.PrefixResourceErrors("storageType", shared.ValidateOptionalInterface(a.StorageType)),
 	))
 }