Skip to content
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

System user and group #104

Merged
merged 2 commits into from
Nov 21, 2015
Merged

System user and group #104

merged 2 commits into from
Nov 21, 2015

Conversation

stintel
Copy link

@stintel stintel commented Nov 20, 2015

It is common practice to use different UID/GID ranges for human users and groups vs users and groups used for services. Adding the users and groups as system users and groups ensures this.

Many distros do this as well when a user or group is created by a package. For example, see https://www.debian.org/doc/debian-policy/ch-opersys.html#s9.2

@TJM
Copy link

TJM commented Nov 21, 2015

I like this, and was gonna file it myself, but was too lazy to do the PR. Thanks!

carlossg added a commit that referenced this pull request Nov 21, 2015
@carlossg carlossg merged commit 3a56c91 into voxpupuli:master Nov 21, 2015
@stintel stintel deleted the system_user_and_group branch November 21, 2015 16:05
@TJM
Copy link

TJM commented May 31, 2017

3 years ago, I thought it was the rvm user itself (doesn't appear to be one). I didn't fully realize that it was the users I was trying to add to the rvm group. It was my bad for not paying attention, but thank you for having the "$create" parameter to turn it off. I was already managing that user anyhow, just that previously we both had ensure_resource, and they were "close enough" to match before. ;)

It's not that I don't agree with the change, but in my lack of sleep induced madness it saddened me enough to reply.

I am actually sortof re-thinking whether the "railsapp" user should be considered a "system" account or not, but for now I am stuck in shibboleth land, pray for me. :)

~tommy

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

Successfully merging this pull request may close these issues.

3 participants