Skip to content

Commit 850ff3a

Browse files
committed
Implement webhook validations for the XGBoostJob
Signed-off-by: Yuki Iwai <[email protected]>
1 parent 1bf512e commit 850ff3a

File tree

10 files changed

+429
-318
lines changed

10 files changed

+429
-318
lines changed

manifests/base/webhook/manifests.yaml

+20
Original file line numberDiff line numberDiff line change
@@ -24,3 +24,23 @@ webhooks:
2424
resources:
2525
- pytorchjobs
2626
sideEffects: None
27+
- admissionReviewVersions:
28+
- v1
29+
clientConfig:
30+
service:
31+
name: webhook-service
32+
namespace: system
33+
path: /validate-kubeflow-org-v1-xgboostjob
34+
failurePolicy: Fail
35+
name: validator.xgboostjob.training-operator.kubeflow.org
36+
rules:
37+
- apiGroups:
38+
- kubeflow.org
39+
apiVersions:
40+
- v1
41+
operations:
42+
- CREATE
43+
- UPDATE
44+
resources:
45+
- xgboostjobs
46+
sideEffects: None

manifests/base/webhook/patch.yaml

+3
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
- op: replace
22
path: /webhooks/0/clientConfig/service/name
33
value: training-operator
4+
- op: replace
5+
path: /webhooks/1/clientConfig/service/name
6+
value: training-operator
47
- op: replace
58
path: /metadata/name
69
value: validator.training-operator.kubeflow.org

pkg/apis/kubeflow.org/v1/xgboost_validation.go

-87
This file was deleted.

pkg/apis/kubeflow.org/v1/xgboost_validation_test.go

-222
This file was deleted.

pkg/cert/cert.go

+2-1
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,10 @@ package cert
1818

1919
import (
2020
"fmt"
21+
"os"
22+
2123
cert "github.com/open-policy-agent/cert-controller/pkg/rotator"
2224
"k8s.io/apimachinery/pkg/types"
23-
"os"
2425
ctrl "sigs.k8s.io/controller-runtime"
2526
)
2627

0 commit comments

Comments
 (0)