dj-wasabi / ansible-telegraf

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

Fixed issues with Windows paths containing spaces & removed redundant "telegraf" folder in path #144

Closed treanorjp closed 3 years ago

treanorjp commented 3 years ago

Description of PR In the Windows install playbook, paths containing spaces would cause the service start task to fail. Wrapped the paths in quotes to fix it. Also the path generated on the host was c:\telegraf\telegraf\telegraf.exe. Changed to c:\telegraf\telegraf.exe

Type of change

Bugfix Pull Request

dj-wasabi commented 3 years ago

Thanks! 👍