search

stakater / application

Generic helm chart for all kind of applications
http://stakater.com
MIT License
213 stars 88 forks source link

feat(cronjob): Corrected indentation of securityContext container params #minor #354

Closed epm-marcus closed 1 week ago

epm-marcus commented 1 week ago

Corrected the indentation level of securityContext to instead apply it to each container within the jobs.

aslafy-z commented 1 week ago

Why was this PR merged? It is a breaking change. Job can have a pod level or container level security context. I think it should be reverted and a new PR should be emitted to add containerSecurityContext.