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

    https://github.com/apache/spark/pull/20822#discussion_r174527459
  
    --- Diff: 
resource-managers/kubernetes/docker/src/main/dockerfiles/spark/entrypoint.sh ---
    @@ -17,13 +17,15 @@
     #
     
     # echo commands to the terminal output
    -set -ex
    +set -x
     
     # Check whether there is a passwd entry for the container UID
     myuid=$(id -u)
     mygid=$(id -g)
     uidentry=$(getent passwd $myuid)
     
    +set -e
    --- End diff --
    
    it is probably best to limit the "no -e" specifically to `getent`, such as:
    ```bash
    set -ex
    myuid=$(id -u)
    mygid=$(id -g)
    # turn off -e for getent because it will return error code in anonymous uid 
case
    set +e
    uidentry=$(getent passwd $myuid)
    set -e
    ```


---

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

Reply via email to