search

dj-wasabi / ansible-telegraf

Installing and configuring Telegraf via Ansible for RedHat/Debian/Ubuntu/Windows/Suse.
MIT License
134 stars 116 forks source link

use version_compare filter … #14

Closed lhoss closed 7 years ago

lhoss commented 7 years ago

…(more readable, stable and no extra set_fact required)

ps: more 'useful' PR coming (adding new agent settings 👍 )

lhoss commented 7 years ago

ps: Changes were already tested (of course I'm using ansible >v2 )

and the travis build did not fail due to my change:

0.25s$ molecule test
ERROR: Driver missing, install docker-py.
dj-wasabi commented 7 years ago

About Travis, I know. I'm fixing it now.

dj-wasabi commented 7 years ago

And thanks!

lhoss commented 7 years ago

wow thx for fast merging ... I was about to write the 'requirements.yml' now I can switch back to the original role 👍