Use ErrorList for validator

Signed-off-by: Kun Chang <curtis@mail.ustc.edu.cn>
kubeflow · Jun 20, 2024 · 7c19c45 · 7c19c45
1 parent f8b8d8d
commit 7c19c45
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)
 	}