Use ErrorList for experiment validator (#2329)

Signed-off-by: Kun Chang <[email protected]>
kubeflow · Jun 27, 2024 · e83628b · e83628b
1 parent 57ed828
commit e83628b
Show file tree

Hide file tree

Showing 3 changed files with 369 additions and 196 deletions.
diff --git a/pkg/webhook/v1beta1/experiment/validation_webhook.go b/pkg/webhook/v1beta1/experiment/validation_webhook.go
@@ -90,7 +90,8 @@ func (v *ExperimentValidator) Handle(ctx context.Context, req admission.Request)
 		return admission.Errored(http.StatusBadRequest, err)
 	}
 
-	err = v.ValidateExperiment(inst, oldInst)
+	allErrs := v.ValidateExperiment(inst, oldInst)
+	err = allErrs.ToAggregate()
 	if err != nil {
 		return admission.Errored(http.StatusBadRequest, err)
 	}