Add a parameter for installing additional packages (postfix-ldap on Fedora-based OSes)

[atcorduneanu]

I am using this role to manage the configuration of my Postfix instances on Fedora-based OSes.
My deployment uses ldap as a backend and ldap support in my case is only available via the RPM postfix-ldap, which currently I am making sure to have it installed in the play's pre_tasks. In this case there is a duplication of the dnf modules called, which is not really a big deal, but I would rather have a variable that if set allows the user to specify which packages to install, or if they need ldap support.

I guess it would be useful for other people as well. Is this a planned feature? Otherwise I would happily work on it.

[richm]

I am using this role to manage the configuration of my Postfix instances on Fedora-based OSes. My deployment uses ldap as a backend and ldap support in my case is only available via the RPM postfix-ldap, which currently I am making sure to have it installed in the play's pre_tasks. In this case there is a duplication of the dnf modules called, which is not really a big deal, but I would rather have a variable that if set allows the user to specify which packages to install, or if they need ldap support.

I guess it would be useful for other people as well. Is this a planned feature? Otherwise I would happily work on it.

It is not planned, but it sounds useful. I'll note that some of the system roles have a variable ROLENAME_optional_packages: [] and some have ROLENAME_additional_packages: [] defined in the defaults/vars.yml - I'll leave it up to you which one makes the most sense for this case.

If you submit a PR, please update the README.md, and please add a test, or extend an existing test, to cover this case.