Skip to content

Commit 3249182

Browse files
author
Martin Linkhorst
committed
move ALBC setting into admission configuration to avoid circular dependency
1 parent 91e9452 commit 3249182

File tree

2 files changed

+3
-8
lines changed

2 files changed

+3
-8
lines changed

cluster/config-defaults.yaml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -714,13 +714,6 @@ teapot_admission_controller_enable_write_protection_webhook: "true"
714714
# configure the behaviour of the resource protection admission webhook, `true` blocks, `false` allows
715715
teapot_admission_controller_prevent_write_operations: "true"
716716

717-
# inject default values into Services for ALBC on EKS
718-
{{ if and (eq .Cluster.Provider "zalando-eks") (eq .Cluster.ConfigItems.eks_ip_family "ipv6") }}
719-
teapot_admission_controller_inject_albc_defaults: "true"
720-
{{ else }}
721-
teapot_admission_controller_inject_albc_defaults: "false"
722-
{{ end }}
723-
724717
# Enable and configure Pod Security Policy rules implemented in admission-controller.
725718
teapot_admission_controller_pod_security_policy_enabled: "true"
726719

cluster/manifests/02-admission-control/config.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,9 @@ data:
1313
dns.default.subdomain-max-length: "{{ .Cluster.ConfigItems.subdomain_max_length }}"
1414

1515
generic.prevent-write-operations.enable: "{{ .Cluster.ConfigItems.teapot_admission_controller_prevent_write_operations }}"
16-
generic.inject-albc-defaults.enable: "{{ .Cluster.ConfigItems.teapot_admission_controller_inject_albc_defaults }}"
16+
{{- if and (eq .Cluster.Provider "zalando-eks") (eq .Cluster.ConfigItems.eks_ip_family "ipv6") }}
17+
generic.inject-albc-defaults.enable: "true"
18+
{{- end }}
1719

1820
pod.container-resource-control.min-memory-request: "25Mi"
1921
pod.container-resource-control.default-cpu-request: "{{ .Cluster.ConfigItems.teapot_admission_controller_default_cpu_request }}"

0 commit comments

Comments
 (0)