openstack/puppet-swift
Code Issues Proposed changes

Remove creation of /home/swift

Browse Source 
It isn't used in the puppet module anywhere, the signing directory is
defaulting to /var/cache/swift, if the user is setting it to /home/swift
they should also create it.

Change-Id: I085bdba172b6316f9dcd19dcc8d44399be014647
This commit is contained in:
Derek Higgins
2015-04-22 15:47:59 +01:00
parent 35b386479d
commit b0a3d8a298
2 changed files with 0 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
manifests/init.pp
View File

@@ -49,11 +49,6 @@ class swift(
File { owner => 'swift', group => 'swift', require => Package['swift'] } File { owner => 'swift', group => 'swift', require => Package['swift'] }
file { '/home/swift':
ensure => directory,
mode => '0700',
}
file { '/etc/swift': file { '/etc/swift':
ensure => directory, ensure => directory,
mode => '2770', mode => '2770',

4
spec/classes/swift_spec.rb
View File

@@ -32,10 +32,6 @@ describe 'swift' do
} }
end end
it {is_expected.to contain_user('swift')} it {is_expected.to contain_user('swift')}
it {is_expected.to contain_file('/home/swift').with(
{:ensure => 'directory', :mode => '0700'
}.merge(file_defaults)
)}
it {is_expected.to contain_file('/etc/swift').with( it {is_expected.to contain_file('/etc/swift').with(
{:ensure => 'directory', :mode => '2770' {:ensure => 'directory', :mode => '2770'
}.merge(file_defaults) }.merge(file_defaults)