Skip to content

Conversation

@DerekTBrown
Copy link

What changes were proposed in this pull request? / Why are the changes needed?

  • Currently, the official Spark docker images run as the UID 185, but no corresponding entry exists in /etc/passwd. This causes issues when libraries try to fetch the current unix username.
  • This PR invokes the useradd function as part of the Dockerfile, to ensure that a name exists for these users.

Does this PR introduce any user-facing change?

  • No. No user-facing changes are expected.

How was this patch tested?

  • Docker build succeeds locally.

@HyukjinKwon
Copy link
Member

@DerekTBrown Mind creating a JIRA and add it to the PR title please? See also https://spark.apache.org/contributing.html

@DerekTBrown DerekTBrown changed the title fix: name docker users SPARK-43112: name docker users Apr 17, 2023
@DerekTBrown DerekTBrown changed the title SPARK-43112: name docker users SPARK-43166: name docker users Apr 17, 2023
@pan3793
Copy link
Member

pan3793 commented Apr 19, 2023

@DerekTBrown I have an alternative solution #40831, which should cover your case

@DerekTBrown
Copy link
Author

Looks good. Closing in favor of #40831

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants