Skip to content

Commit

Permalink
skipper: add config to enable ingress validation webhook
Browse files Browse the repository at this point in the history
Followup on #6341

Signed-off-by: Alexander Yastrebov <[email protected]>
  • Loading branch information
AlexanderYastrebov committed Aug 24, 2023
1 parent 1b0d57e commit e758285
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
3 changes: 3 additions & 0 deletions cluster/config-defaults.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -246,6 +246,9 @@ skipper_serve_status_code_metric: "false"
# can be one of disabled|provisioned|enabled
routegroups_validation: "enabled"

# disabled|enabled ingress validation ( skipper webhook )
ingresses_validation: "disabled"

# tokeninfo
{{if eq .Cluster.Environment "production"}}
# production|bridge|disabled
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
application: skipper-ingress
component: webhook
webhooks:
{{ if eq .Cluster.ConfigItems.routegroups_validation "enabled" }}
# {{ if eq .Cluster.ConfigItems.routegroups_validation "enabled" }}
- name: "routegroup-admitter.teapot.zalan.do"
rules:
- operations: ["CREATE", "UPDATE"]
Expand All @@ -19,7 +19,8 @@ webhooks:
admissionReviewVersions: ["v1"]
sideEffects: None
timeoutSeconds: 5
{{ end }}
# {{ end }}
# {{ if eq .Cluster.ConfigItems.ingresses_validation "enabled" }}
- name: "ingress-admitter.teapot.zalan.do"
rules:
- operations: ["CREATE", "UPDATE"]
Expand All @@ -31,4 +32,5 @@ webhooks:
caBundle: "{{ .ConfigItems.ca_cert_decompressed }}"
admissionReviewVersions: ["v1"]
sideEffects: None
timeoutSeconds: 5
timeoutSeconds: 5
# {{ end }}

0 comments on commit e758285

Please sign in to comment.