-
-
Notifications
You must be signed in to change notification settings - Fork 82
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
ability to change steam userid #16
Comments
Where are puid and guid variables coming from? |
Same. So what happens is the container needs to know what uid/gid to make the steam:steam user run as... Should pull those from environment variables during setup. So for me I can't get through install even though I have 777 the directories on host system If you look at https://hub.docker.com/u/linuxserver/ they have some good examples of how it should work. |
@MattPark and @Fabiryn I need some help, After doing some digging here are my findings and please correct me if I am wrong before I implement a fix. The This is a security feature of docker limit what can and cant occur in terms of file reads + writes on the host. Does this sound about right? |
Your understanding is correct @mbround18 but to try and add more context: Your base image has two users, Several container creators such as https://github.com/linuxserver/ respect a "PGID" and "PUID" environment variable which will set the UID and GID of that created user. It's still the same "user" within Docker, just mapped to a different PGID/PUID. See https://github.com/linuxserver/docker-baseimage-ubuntu/blob/b529d350b1438aa81e68a5d87eff39ade0f1c879/root/etc/cont-init.d/10-adduser#L6 for an example of how they do this. |
Thank you @vorpalhex and @MattPark, I was lost on this whole permissions thing. I think I have a fix for this but I am out of time to give it a whirl. Ill toss my branch up if one of ya'll do not mind giving it a spin locally? |
See PR #18 for the updates to acknowledge the |
* #16 - Added PUID and PGID environment variables * Disabled auto labeler for now * Fixed timezone issue and updated readme * Readme changes and version fix * Removed Labeler
This issue is fixed, could someone please verify on |
Looking at PR #18, I think you nailed it. I will pull the latest from docker hub and try that to confirm. Nice job on this, especially for a first time.
|
Seems like the docker user and the environment puid pgid are ignored.
currently I can only manage to run the container if I grant 777 access to the mounted volume.
The text was updated successfully, but these errors were encountered: