Github user vanzin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/23017#discussion_r233529869
  
    --- Diff: 
resource-managers/kubernetes/docker/src/main/dockerfiles/spark/entrypoint.sh ---
    @@ -30,6 +30,10 @@ set -e
     # If there is no passwd entry for the container UID, attempt to create one
     if [ -z "$uidentry" ] ; then
         if [ -w /etc/passwd ] ; then
    +        # TODO Should we allow providing an environment variable to set 
the desired username?
    --- End diff --
    
    So aren't you basically saying that the entrypoint script is being run a 
root? Isn't that exactly what we do not want, since that will be running user 
code as root inside the image?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to