ntd / ansible-role-nut

Ansible role to install and configure Nework UPS tools on Debian
MIT License
11 stars 18 forks source link

add upsd.conf.j2. add support for more variables #1

Closed Torstein-Eide closed 4 years ago

Torstein-Eide commented 4 years ago

remoted option to not control config.

ntd commented 4 years ago

Hi @Eideen, thanks for your contribution. Trying to map any NUT option to an ansible variable is highly impractical and in my opinion harmful. Some of the ideas included by this PR were good though.

In the dev branch I included this PR and partially reverted it, using a different approach instead to allow full customization of the configuration. I just added a fistful of ...extra options that are appended verbatim in crucial places.

Let me know if this solves your issues, in which case I'll merge dev into master.