[GitHub] [airflow] nwalens commented on a diff in pull request #24588: add: container securityContext not available in podSecurityContext

[GitBox <gi...@apache.org>]

nwalens commented on code in PR #24588:
URL: https://github.com/apache/airflow/pull/24588#discussion_r947804136


##########
chart/templates/_helpers.yaml:
##########
@@ -703,14 +703,14 @@ Create the name of the cleanup service account to use
 {{- end -}}
 
 {{/*
-Set the default value for securityContext
-If no value is passed for securityContext or <node>.securityContext, defaults to global uid and gid.
+Set the default value for pod securityContext
+If no value is passed for securityContext.pod or <node>.securityContext.pod, defaults to global uid and gid.

Review Comment:
   Hi, I think airflowSecurityContextIds is used on workers-deployment to set the uid and gid during a chown. Not sure if has changed.



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org