-
-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Docker permissions issue #56
Comments
Hey! Just to clear things up, the goal is to give stump only the permissions to the config directory and not create it within the host machine? Can I take on this issue? |
Hi! So to clarify, when the config folder (e.g. version: '3.3'
services:
stump:
image: aaronleopold/stump-preview
ports:
- '10801:10801'
user: '1000:1000'
volumes:
- /Users/aaronleopold/Documents/Stump:/data
- /Users/aaronleopold/.stump:/config When I run If you still would like it, feel free to take it on! Keep in mind, if it is for hacktoberfest don't make the PR until it actually starts. Otherwise, I am not sure it will be counted towards your total. For the quickest debugging, use a variation of |
That makes sense, thank you :) I will still take it on! |
That docker's standard behaviour and is not really fixable afaik. The alternative would be to use a docker volume instead of a host-path-binding. Would that be an acceptable solution? Update: When trying to do the bind-mount, docker-compose actually errors out and won't let the user startup the container
|
…les (#89) * docker: Allowing to downgrade to regular user via environment variables (fixed issue #56) * docker: Use full relative path to entrypoint.sh Relative path as seen from repo-root Co-authored-by: Aaron Leopold <[email protected]> --------- Co-authored-by: Aaron Leopold <[email protected]>
Resolved from #89 |
Docker seems to create stump's config directory, on the host machine, with root privilege, making root the owner. This becomes a big problem during runtime, as stump won't have permissions to modify/access the config folder, which causes a ton of errors.
Changing the permissions manually fixes this, but this is really not ideal.
I have only been able to reproduce this issue on Linux (both a dedicated install, as well as WSL). Config folder must not already be present (
~/.stump
), otherwise issue won't occur.The text was updated successfully, but these errors were encountered: