aws / eks-charts

Amazon EKS Helm chart repository
Apache License 2.0
1.17k stars 922 forks source link

Add selector to exclude namespace admission controller is deployed in #1089

Open paulbraham-ds opened 2 months ago

paulbraham-ds commented 2 months ago

Issue

https://github.com/aws/eks-charts/issues/1088

Description of changes

This PR adds a namespaceSelector so the mutating webhook does not run on the namespace the webhook controller is installed in.

Checklist

Testing

Changes submitted in this PR tested locally against Helm v3 on an EKS 1.27 cluster. After these additions I can delete all pods in the deployment and new pods come back without issue.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

paulbraham-ds commented 1 month ago

Anybody able to review this?