Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/23017#discussion_r233215367
--- 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 --
I think it's ok to remove this TODO. The user name propagation in Hadoop
should be fixed elsewhere (in fact I'm touching that code in another PR...).
But more interesting, by having a default UID, this code becomes a no-op,
since `/etc/passwd` is not writable anymore.
And that's also probably the cause of the test failure you're seeing.
So maybe you need to call `useradd` or whatever is the appropriate command
in this image to create that user when creating the image.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
