Skip to content

Commit

Permalink
Add rvm users as system users
Browse files Browse the repository at this point in the history
  • Loading branch information
stintel committed Nov 20, 2015
1 parent 5985486 commit 77ad90a
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion manifests/system_user.pp
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,10 @@
}

if $create {
ensure_resource('user', $name, {'ensure' => 'present' })
ensure_resource('user', $name, {
'ensure' => 'present',
'system' => true,
})
User[$name] -> Exec["rvm-system-user-${name}"]
}

Expand Down

2 comments on commit 77ad90a

@TJM
Copy link

@TJM TJM commented on 77ad90a May 31, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change burned me. I don't consider RVM users (application users) to be system accounts.

@stintel
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 years ago you liked the change. Also see extra motivation for the change in #104 ...

Please sign in to comment.