Use ErrorList for experiment validator (kubeflow#2329)

Signed-off-by: Kun Chang <curtis@mail.ustc.edu.cn>
shashank-iitbhu · Jun 30, 2024 · cb096ed · cb096ed
1 parent 55de8c4
commit cb096ed
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)
 	}