Aakcht commented on PR #31865: URL: https://github.com/apache/airflow/pull/31865#issuecomment-1601343573
@jedcunningham , please take a look. I completely excluded redis/statsd from global security context. Looks like I was wrong in my previous comment and using localContainerSecurityContext logic for redis/statsd container security context is not the best idea, cause it might lead to some of pod security context parameters ending up in container level context. At the same time looks like simple `.Values.redis.securityContexts.container`/`.Values.redis.securityContexts.container` works well in this case - at least I can't see anything wrong with it except for the fact that it looks too easy :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
