Skip to content
This repository has been archived by the owner on Feb 24, 2022. It is now read-only.

Allow running fpm as root #79

Closed
wants to merge 1 commit into from
Closed

Allow running fpm as root #79

wants to merge 1 commit into from

Conversation

michaeljs1990
Copy link

For one reason or another in my environment it is much easier to run this as root... Mostly due to permissions around my NAS system. This will allow people if they so dare to set the PUID to 0 and run this container.

linuxserver.io

Thanks, team linuxserver.io

For one reason or another in my environment it is much easier to run this as root... Mostly due to permissions around my NAS system. This will allow people if they so dare to set the PUID to 0 and run this container.
@LinuxServer-CI
Copy link
Contributor

Can an admin please whisper to jenkins to test this PR.

@michaeljs1990
Copy link
Author

michaeljs1990 commented Mar 20, 2018

I have pushed michaeljs1990/rutorrent to my dockerhub as well although it's easy enough to build locally.

Additionally if we are adamantly against running as root for whatever reason I can add update the readme and add some error checking to throw if a PUID of 0 is set. Currently you have to dig around in /var/logs to find what is happening.

@sparklyballs
Copy link
Contributor

we don't recommend running our containers as root and this seems an edge case to me

@michaeljs1990 michaeljs1990 deleted the patch-1 branch March 20, 2018 15:33
@soruly
Copy link

soruly commented Dec 17, 2019

When running with podman (rootless container), if we define PUID=1000, the UID would be run as 100999 on host, which is incorrect. In order to correctly map it, we need to run as PUID=0 which maps correctly to host uid who starts the container.
https://discuss.linuxcontainers.org/t/looking-for-clarification-about-subuid-gid/630/3

libpod actually recommends running containers with UID=0
containers/podman#3421 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants