openstack/puppet-openstack-integration
Code Issues Proposed changes

Debian: correct /var/log/swift unix rights

Browse Source 
Debian package uses swift:adm with 0750, let's not force something else,
as this makes the puppet run not indempotent.

Change-Id: I325e2f722b11deb51a1b8f40d21fddac4593834c
This commit is contained in:
Thomas Goirand 2018-05-16 15:05:02 +02:00
parent 4bbea36c6f
commit 423c5d0b5e
1 changed files with 11 additions and 4 deletions

15
manifests/swift.pp

@ -13,9 +13,16 @@ class openstack_integration::swift {
require => Package['rsyslog'],
}
file { '/var/log/swift':
ensure => directory,
mode => '0755',
if ($::os_package_type == 'debian') {
file { '/var/log/swift':
ensure => directory,
mode => '0750',
}
} else {
file { '/var/log/swift':
ensure => directory,
mode => '0755',
}
}
file { '/etc/rsyslog.d/10-swift.conf':
ensure => present,
@ -28,7 +35,7 @@ class openstack_integration::swift {
if $::osfamily == 'Debian' {
if $::os_package_type == 'debian' {
File<| title == '/var/log/swift' |> {
owner => 'root',
owner => 'swift',
group => 'adm'
}
} else {