[BUG][FEATURE] service management needs improvement

[daks]

Your setup

Formula commit hash / release tag

c0485fb345bbbdbbf96e20653f2df36d85910237 26/10/2020

Versions reports (master & minion)

not relevant

Pillar / config used

neither

---

Bug details

Describe the bug

Different problems about service management in this formula

• no state to make a systemctl reload postfix, it's only possible to make a restart, see https://github.com/saltstack-formulas/postfix-formula/blob/master/postfix/init.sls#L36
• two states in init.sls to manage the postfix service, one here https://github.com/saltstack-formulas/postfix-formula/blob/master/postfix/init.sls#L25 another https://github.com/saltstack-formulas/postfix-formula/blob/master/postfix/init.sls#L36
• the state L25 uses a reload parameter which, according to doc and also salt code, does not exist

Steps to reproduce the bug

Expected behaviour

I want to be able to make a reload not a restart of the service. It's useful for example when changing a file for mynetworks parameter. A restart introduces a small service interruption when it's not necessary.

Attempts to fix the bug

None

Additional context

[daks]

Looks like I was wrong about the reload part, it's not visible in doc or code because it's managed by mod_watch in the same state module https://docs.saltstack.com/en/latest/ref/states/all/salt.states.service.html#salt.states.service.mod_watch

[fzipi]

@daks Can this one be closed then?