securityContext duplicated in binderhub/helm-chart/binderhub/templates/container-builder/daemonset.yaml
rgaiacs opened this issue · 1 comments
rgaiacs commented
securityContext is declared at
and
Should we remove the first occurence?
sgaist commented
Hi,
No, it's on purpose. Each security context applies to the corresponding builder. In this case, you can see that the Podman builder has an additional requirement which might change in the future and does not concern the Docker builder. It's better to have that field tailored to the container definition rather than making it generic for all.
This can of course change in a future revision if all builders make use of the same security context.
Best regards