Skip to content

Commit

Permalink
Fix nologin shell path on Debian
Browse files Browse the repository at this point in the history
We don't use /usr/sbin/nologin which works on Debian but breaks EL6.
  • Loading branch information
sathieu committed Apr 22, 2014
1 parent 99f032b commit dcb8a70
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion manifests/config.pp
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@

user { $foreman::user:
ensure => 'present',
shell => '/sbin/nologin',
shell => '/bin/false',
comment => 'Foreman',
home => $foreman::app_root,
gid => $foreman::group,
Expand Down
4 changes: 2 additions & 2 deletions spec/classes/foreman_config_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@

it { should contain_user('foreman').with({
'ensure' => 'present',
'shell' => '/sbin/nologin',
'shell' => '/bin/false',
'comment' => 'Foreman',
'gid' => 'foreman',
'groups' => ['puppet'],
Expand Down Expand Up @@ -201,7 +201,7 @@

it { should contain_user('foreman').with({
'ensure' => 'present',
'shell' => '/sbin/nologin',
'shell' => '/bin/false',
'comment' => 'Foreman',
'gid' => 'foreman',
'groups' => ['puppet'],
Expand Down

0 comments on commit dcb8a70

Please sign in to comment.