Signed-off-by: Felipe Zipitria <fzipitria@perceptyx.com>
BREAKING CHANGE: `transport` and `tls_policy` keys in `postfix:config` have been moved to the standard `postfix:mapping`.
* Semi-automated using `ssf-formula` (v0.5.0)
* Fix (or ignore) errors shown below:
```bash
postfix-formula$ $(grep "\- yamllint" .travis.yml | sed -e "s:^\s\+-\s\(.*\):\1:")
./postfix/osfamilymap.yaml
1:1 warning missing document start "---" (document-start)
./postfix/services.yaml
11:1 warning missing document start "---" (document-start)
12:1 warning comment not indented like content (comments-indentation)
221:1 warning comment not indented like content (comments-indentation)
./postfix/defaults.yaml
4:1 warning missing document start "---" (document-start)
pillar.example
1:1 warning missing document start "---" (document-start)
2:25 warning truthy value should be one of [false, true] (truthy)
13:17 warning truthy value should be one of [false, true] (truthy)
16:17 warning truthy value should be one of [false, true] (truthy)
22:17 warning truthy value should be one of [false, true] (truthy)
23:17 warning truthy value should be one of [false, true] (truthy)
25:17 warning truthy value should be one of [false, true] (truthy)
29:17 warning truthy value should be one of [false, true] (truthy)
31:17 warning truthy value should be one of [false, true] (truthy)
35:17 warning truthy value should be one of [false, true] (truthy)
47:15 warning truthy value should be one of [false, true] (truthy)
50:18 warning truthy value should be one of [false, true] (truthy)
53:21 warning truthy value should be one of [false, true] (truthy)
62:24 warning truthy value should be one of [false, true] (truthy)
66:31 warning truthy value should be one of [false, true] (truthy)
69:19 warning truthy value should be one of [false, true] (truthy)
70:19 warning truthy value should be one of [false, true] (truthy)
73:14 warning truthy value should be one of [false, true] (truthy)
74:21 warning truthy value should be one of [false, true] (truthy)
78:14 warning truthy value should be one of [false, true] (truthy)
89:15 error trailing spaces (trailing-spaces)
96:6 warning missing starting space in comment (comments)
130:89 error line too long (105 > 88 characters) (line-length)
131:89 error line too long (101 > 88 characters) (line-length)
152:5 error duplication of key "sender_canonical_maps" in mapping (key-duplicates)
154:5 error duplication of key "virtual_alias_maps" in mapping (key-duplicates)
190:5 error duplication of key "use_file" in mapping (key-duplicates)
235:9 error wrong indentation: expected 10 but found 8 (indentation)
248:1 error duplication of key "postfix" in mapping (key-duplicates)
258:85 error trailing spaces (trailing-spaces)
```
Add dovecot example using the services dictionary with a note that
LMTP delivery might be better.
Slight improvements about pillar example settings.
Reorder examples, put the services dictionary examples which allow
more control above the previous dovecot and submission examples.
Add a note about backwards compatibility to indicate that these
options are still supported.
As promised in PR #75, here's a further (and hopefully final)
iteration of the master.cf service handling:
- Bring dovecot and policyd-spf configuration in line with the
extras_services item added in PR #75
- Remove policyd-spf configuration again from services.jinja,
handle this in master.cf instead, where it was before
- Allow for completely custom services to be added via a pillar
definition in postfix:master_config:services
- Fix legacy dovecot argv example in pillar.example: The example
showed the extra_args being defined as part of the argv
parameter, something that would clash with the actual code and
result in a duplicate '-d ${recipient}' definition in the config.
Most of the services in master.cf were already configurable through
the formula but specific ones such as cyrus or uucp were not managed
yet.
This commit adds functionality to manage these services as well
through pillar variables, e.g.:
postfix:
master_config:
services:
uucp:
enable: True
The postfix service is currently being restarted whenever a
config item changes.
This is unnecessary as the postfix service can reload a new
config.
Set reload: True to prevent unnecessary restarts of the service.
The old behavior can be restored by setting the pillar key
reload_service to False.
Currently master.cf only allows for _very_ limited configuration
options mainly focussed on SMTP submission settings.
This is rather limited and does not scale very well for managing
the other services defined in master.cf.
This patch has moved all the service definitions into a jinja file
and generates the master.cf service definition on the fly based on
these defaults.
Defaults can be overridden in a pillar to customize the rendered
master.cf file accordingly to local needs.
Undefined values will be filled with the postfix defaults.
Care has been taken that the previous ways of managing the submission
configuration options are still supported for backwards compatibility
to prevent breakage for existing users of the formula.
Merge existing handling of postfix lookup table/map files into a single
template. Mappings are read from pillar in `postfix:mapping`.
Configuration is written to the file pointed to by the relevant
directive in `postfix:config`. A single target file is supported at the
moment. The file is postmap'ed if needed.
The pillar accepts a dict or an OrderedDict.
Also change path to certificates since previous ones are distribution
specific. They look like Debian path, Gentoo uses different ones.
New path uses same logic as nginx's formula, use known to exist folder
which server most likely has permission to read too since it is its
configuration folder.
