CAAPIM / apim-charts

Helm Charts for Layer7 API Management components.
MIT License
11 stars 48 forks source link

[charts/gateway] Update otk-install-job.yaml with ContainerSecurityContext from gateway values chart #232

Closed SebastiaanKveek closed 1 year ago

SebastiaanKveek commented 1 year ago

Description of the change

Currently there is a ContainerSecurityContext option in the main values file. It is expected that this is applied to all containers deployed with the gateway helm chart. However it is only applied to the deployment template and the pm-tagger-deployment template (which also seems bugged). Before I would patch this with using kustomize as a post renderer but with this particular job that is not possible since it's a post-install,post-upgrade helm job.

Benefits

You will be able to set the containerSecurityContext to meet your kubernetes cluster's security requirements and run the job.

Drawbacks

No drawbacks since it is optional.

Applicable issues

Additional information Currently I cannot use the otk-install job since our cluster has Polaris running which blocks any containers without a specific SecurityContext defined.

Checklist

Gazza7205 commented 1 year ago

Thank you @SebastiaanKveek - this change has been added to this PR

SebastiaanKveek commented 1 year ago

Thanks!