You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "DerekTBrown (via GitHub)" <gi...@apache.org> on 2023/04/14 21:55:08 UTC
[GitHub] [spark] DerekTBrown opened a new pull request, #40798: fix: name docker users
DerekTBrown opened a new pull request, #40798:
URL: https://github.com/apache/spark/pull/40798
### 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](https://stackoverflow.com/questions/41864985/hadoop-ioexception-failure-to-login) 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.
--
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: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] DerekTBrown commented on pull request #40798: SPARK-43166: name docker users
Posted by "DerekTBrown (via GitHub)" <gi...@apache.org>.
DerekTBrown commented on PR #40798:
URL: https://github.com/apache/spark/pull/40798#issuecomment-1520512047
Looks good. Closing in favor of #40831
--
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: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] HyukjinKwon commented on pull request #40798: fix: name docker users
Posted by "HyukjinKwon (via GitHub)" <gi...@apache.org>.
HyukjinKwon commented on PR #40798:
URL: https://github.com/apache/spark/pull/40798#issuecomment-1510632838
@DerekTBrown Mind creating a JIRA and add it to the PR title please? See also https://spark.apache.org/contributing.html
--
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: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] pan3793 commented on pull request #40798: SPARK-43166: name docker users
Posted by "pan3793 (via GitHub)" <gi...@apache.org>.
pan3793 commented on PR #40798:
URL: https://github.com/apache/spark/pull/40798#issuecomment-1514545693
@DerekTBrown I have an alternative solution https://github.com/apache/spark/pull/40831, which should cover your case
--
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: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
[GitHub] [spark] DerekTBrown closed pull request #40798: SPARK-43166: name docker users
Posted by "DerekTBrown (via GitHub)" <gi...@apache.org>.
DerekTBrown closed pull request #40798: SPARK-43166: name docker users
URL: https://github.com/apache/spark/pull/40798
--
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: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org