OSC/puppet-module-openondemand

Explicitly declare permissions for config directory

Closed this issue · 1 comments

Relevant code:

file { '/etc/ood/config/apps':
ensure => 'directory',
owner => 'root',
group => 'root',
source => $openondemand::_apps_config_source,
recurse => true,
purge => true,
force => true,
}

This breaks if the target system has a root umask without world-execute.

My suggestion is to add

mode => '0755',

I ran into this issue while trying to puppetize https://osc.github.io/ood-documentation/latest/customizations.html#add-shortcuts-to-files-menu. I can fix the permissions of the subdirectories but not this one, since it's already declared.

This is fixed in v4.1.0. The mode is set to 0644 because Puppet automatically adds execute bit to directories and this ensures execute bit is not added to files as part of the recursive management.