Add ability to pass 'dnsConfig' and 'securityContext' attributes in the 'operator' deployment resource.#2069
Conversation
420116b to
62fc72b
Compare
|
/ok-to-test 62fc72b |
|
@mihkelparna1 Would you be able to address the review comments so we can get this PR merged soon? Thanks! |
…he 'operator' deployment resource. Signed-off-by: Mihkel Pärna <mihkel.parna@wise.com> Signed-off-by: Mihkel Pärna <mihkel.parna@transferwise.com>
62fc72b to
064b47b
Compare
|
/ok-to-test 064b47b |
Testing stepsAdditive dnsConfigAdds search domains and a custom nameserver on top of the cluster DNS. dnsPolicy dnsPolicy: NoneForces the pod to ignore cluster DNS entirely. Useful to confirm the new dnsPolicy |
|
Thanks for the PR @mihkelparna1. Is this change required only for gpu-operator deployment? We have a bunch of other operands provisioned by gpu-operator once installed, so they should also get blocked if kyverno/OPA wants them to be set before admitting resources. Are you not seeing them blocked in your case? Trying to make sense as to why only gpu-operator deployment (seen during helm rendering) needs this and not the rest of the operands (rendered later on by gpu-operator pod). |
Fixes #2070
Description
Checklist
make lint)make validate-generated-assets)make validate-modules)Testing
Tested out by rendering out all options ( Fields set, fields unset, rendering both options )