RexXiong commented on code in PR #3251:
URL: https://github.com/apache/celeborn/pull/3251#discussion_r2081849949


##########
charts/celeborn/ci/values.yaml:
##########
@@ -139,14 +148,14 @@ worker:
   # -- Whether to use the host's network namespace in Celeborn worker pods.
   hostNetwork: true
 
-# -- Container security context
-securityContext:
-  # Specifies the user ID to run the entrypoint of the container process
-  runAsUser: 10006
-  # Specifies the group ID to run the entrypoint of the container process
-  runAsGroup: 10006
-  # Specifies the group ID to use when modifying ownership and permissions of 
the mounted volumes
-  fsGroup: 10006
+  # -- Pod-level security configurations for Celeborn worker pods.
+  podSecurityContext:

Review Comment:
   Do we still need to keep the securityContext in the values file?



-- 
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]

Reply via email to