kosmos/postfix-cookbook
8
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix OpenSUSE installation issues

Browse Source 
OpenSUSE runs a script which builds the main.cf file automatically if a file
doesn't exist. This breaks idempotency so go ahead and touch the file via Chef.

In addition, postfix is built with incorrect default settings for `setgid_group`
and `daemon_directory`, so set it properly in the config.

Signed-off-by: Lance Albertson <lance@osuosl.org>
This commit is contained in:
Lance Albertson 2020-10-11 10:44:22 -07:00
parent 3727f97797
commit 7208c3f559
3 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
CHANGELOG.md
View File

@ -26,6 +26,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- ChefSpec fixes - ChefSpec fixes
- Yamllint fixes - Yamllint fixes
- MDL fixes - MDL fixes
- Fix OpenSUSE installation issues
### Removed ### Removed

3
attributes/default.rb
View File

@ -105,6 +105,9 @@ when 'rhel'
default['postfix']['cafile'] = '/etc/pki/tls/cert.pem' default['postfix']['cafile'] = '/etc/pki/tls/cert.pem'
when 'amazon' when 'amazon'
default['postfix']['cafile'] = '/etc/pki/tls/cert.pem' default['postfix']['cafile'] = '/etc/pki/tls/cert.pem'
when 'suse'
default['postfix']['main']['setgid_group'] = 'maildrop'
default['postfix']['main']['daemon_directory'] = '/usr/lib/postfix/bin'
else else
default['postfix']['cafile'] = "#{node['postfix']['conf_dir']}/cacert.pem" default['postfix']['cafile'] = "#{node['postfix']['conf_dir']}/cacert.pem"
end end

2
recipes/_common.rb
View File

@ -42,6 +42,8 @@ when 'rhel', 'fedora', 'amazon'
notifies :start, 'service[postfix]' notifies :start, 'service[postfix]'
not_if '/usr/bin/test /etc/alternatives/mta -ef /usr/sbin/sendmail.postfix' not_if '/usr/bin/test /etc/alternatives/mta -ef /usr/sbin/sendmail.postfix'
end end
when 'suse'
file '/var/adm/postfix.configured'
when 'omnios' when 'omnios'
manifest_path = ::File.join(Chef::Config[:file_cache_path], 'manifest-postfix.xml') manifest_path = ::File.join(Chef::Config[:file_cache_path], 'manifest-postfix.xml')